add robots.txt, introduce first search draft

.mvn/wrapper/maven-wrapper.properties

@@ -14,5 +14,5 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
-distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.4/apache-maven-3.9.4-bin.zip
+distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.5/apache-maven-3.9.5-bin.zip
 wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.2.0/maven-wrapper-3.2.0.jar

README.md

@@ -24,7 +24,7 @@ Apache License 2.0 ([Apache-2.0](https://www.apache.org/licenses/LICENSE-2.0))
 * JDK 17+
 * MySQL 8+
 
-Just put the [JDK](https://adoptium.net/temurin/releases/) somewhere on your file system.
+Just put the [JDK](https://adoptium.net/temurin/releases/?os=any&arch=any&package=jdk&version=17) somewhere on your file system.
 The `bin` folder contains the `java` binary.
 
 ### How to configure

backend/src/main/java/com/github/binpastes/config/WebServerConfig.java

@@ -49,6 +49,8 @@ public RouterFunctionMapping indexRoute(@Value("static/index.html") final ClassP
 
         var route = route(RequestPredicates
                         .method(HttpMethod.GET)
+                        .and(path("/robots.txt").negate())
+                        .and(path("/favicon.png").negate())
                         .and(path("/assets/**").negate())
                         .and(path("/api/**").negate()),
                 request -> ok().contentType(MediaType.TEXT_HTML).bodyValue(indexHtml));

backend/src/main/java/com/github/binpastes/paste/api/PasteController.java

@@ -29,8 +29,8 @@
 
 import static com.github.binpastes.paste.api.model.ListView.ListItemView;
 
-@RestController
 @Validated
+@RestController
 @RequestMapping("/api/v1/paste")
 class PasteController {
 
@@ -79,11 +79,11 @@ public Mono<ListView> findPastes() {
     public Mono<SearchView> searchPastes(
             @RequestParam("term")
             @NotBlank
-            @Pattern(regexp = "[\\pL\\pN\\s]{3,25}")
+            @Pattern(regexp = "[\\pL\\pN\\p{P}\\s]{3,25}")
             final String term,
             final ServerHttpResponse response
     ) {
-        response.getHeaders().add(HttpHeaders.CACHE_CONTROL, "max-age=300");
+        response.getHeaders().add(HttpHeaders.CACHE_CONTROL, "max-age=60");
         return pasteService
                 .findByFullText(term)
                 .map(paste -> SearchItemView.of(paste, term))

backend/src/main/java/com/github/binpastes/paste/api/model/SearchView.java

@@ -22,7 +22,7 @@ public record SearchItemView(
             LocalDateTime dateOfExpiry
     ) {
 
-        private static final short HIGHLIGHT_RANGE = 25;
+        private static final short HIGHLIGHT_RANGE = 30;
 
         public static SearchItemView of(final Paste reference, final String term) {
             return new SearchItemView(

backend/src/main/java/com/github/binpastes/paste/domain/MySqlFullTextSupportImpl.java

@@ -27,40 +27,27 @@ public MySqlFullTextSupportImpl(final R2dbcEntityTemplate entityTemplate) {
 
     @Override
     public Flux<Paste> searchByFullText(final String text) {
-        /*
-         * Seems not to be supported by dev.miku:r2dbc-mysql
-         * java.lang.IllegalArgumentException: Cannot encode value of type 'class io.r2dbc.spi.Parameters$InParameter'
-         **/
-/*
-        entityTemplate
-                .getDatabaseClient()
-                .sql("SELECT * FROM pastes WHERE (date_of_expiry IS NULL OR date_of_expiry > CURRENT_TIMESTAMP) AND MATCH(title, content) AGAINST(?text IN BOOLEAN MODE)")
-                .bind("text", text + '*'))
-*/
-
-        var connectionFactory = entityTemplate.getDatabaseClient().getConnectionFactory();
-
-        var query = String.format("SELECT * FROM %s WHERE %s = ? AND (%s IS NULL OR %s > ?) AND (MATCH(%s) AGAINST(? IN BOOLEAN MODE) OR (MATCH(%s) AGAINST(? IN BOOLEAN MODE) AND %s IS FALSE)) ORDER BY %s DESC",
+        var query = String.format("SELECT * FROM %s WHERE %s = ? AND (%s IS NULL OR %s > ?) AND (MATCH(%s) AGAINST(?) OR (%s IS FALSE AND MATCH(%s) AGAINST(?)))",
                 PasteSchema.TABLE_NAME,
                 PasteSchema.EXPOSURE,
                 PasteSchema.DATE_OF_EXPIRY,
                 PasteSchema.DATE_OF_EXPIRY,
                 PasteSchema.TITLE,
-                PasteSchema.CONTENT,
                 PasteSchema.IS_ENCRYPTED,
                 PasteSchema.DATE_CREATED
         );
 
+        var connectionFactory = entityTemplate.getDatabaseClient().getConnectionFactory();
         return Mono.from(connectionFactory.create())
                 .flatMap(mySqlConnection -> Mono.from(mySqlConnection
-                        .createStatement(query)
-                        .bind(0, PasteExposure.PUBLIC.name())
-                        .bind(1, LocalDateTime.now())
-                        .bind(2, text + '*')
-                        .bind(3, text + '*')
-                        .execute()
+                            .createStatement(query)
+                            .bind(0, PasteExposure.PUBLIC)
+                            .bind(1, LocalDateTime.now())
+                            .bind(2, text)
+                            .bind(3, text)
+                            .execute()
                 ))
-                .flatMapMany(mySqlResult -> Flux.from(mySqlResult.map((row, rowMetadata) -> {
+                .flatMapMany(mySqlResult -> Flux.from(mySqlResult.map((row) -> {
                     var paste = new Paste();
                     paste.setId(row.get(PasteSchema.ID, String.class));
                     paste.setVersion(row.get(PasteSchema.VERSION, Long.class));

backend/src/main/java/com/github/binpastes/paste/domain/SimpleFullTextSupportImpl.java

@@ -26,7 +26,7 @@ public SimpleFullTextSupportImpl(final R2dbcEntityTemplate entityTemplate) {
     @Override
     public Flux<Paste> searchByFullText(final String text) {
         var criteria = Criteria
-                .where(PasteSchema.EXPOSURE).is(PasteExposure.PUBLIC.name())
+                .where(PasteSchema.EXPOSURE).is(PasteExposure.PUBLIC)
                 .and(Criteria
                         .where(PasteSchema.DATE_OF_EXPIRY).isNull()
                         .or(PasteSchema.DATE_OF_EXPIRY).greaterThan(LocalDateTime.now())
@@ -47,6 +47,5 @@ public Flux<Paste> searchByFullText(final String text) {
                                 .sort(Sort.by(Sort.Direction.DESC, PasteSchema.DATE_CREATED))
                 )
                 .all();
-
     }
 }

backend/src/main/resources/application-dev.properties

@@ -4,8 +4,6 @@
 #logging.level.org.flywaydb.core.internal.license.VersionPrinter=INFO
 logging.level.com.github.binpastes.paste.business.tracking=DEBUG
 
-spring.jpa.database-platform=org.hibernate.dialect.H2Dialect
-
 spring.flyway.enabled=true
 spring.flyway.driver-class-name=org.h2.Driver
 spring.flyway.fail-on-missing-locations=true

backend/src/main/resources/application-mysql.properties

@@ -1,5 +1,3 @@
-spring.jpa.database-platform=org.hibernate.dialect.MySqlDialect
-
 spring.flyway.enabled=true
 spring.flyway.driver-class-name=com.mysql.cj.jdbc.Driver
 spring.flyway.fail-on-missing-locations=true

frontend/index.html

@@ -8,6 +8,7 @@
   <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
   <meta name="robots" content="noindex, nofollow"/>
   <meta name="viewport" content="width=device-width, initial-scale=1"/>
+  <meta name="color-scheme" content="dark light">
   <link rel="icon shortcut" type="image/png" href="/favicon.png"/>
 </head>
 <body id="root">

frontend/public/robots.txt

@@ -0,0 +1,2 @@
+User-agent: *
+Disallow: /api/

frontend/src/App.tsx

@@ -23,8 +23,8 @@ const App: () => JSX.Element = () => {
           <div class={styles.leftContainer}>
           <Routes>
             <Route path="/" component={Create}  />
-            <Route path="/paste/search" component={Search} />
             <Route path="/paste/:id" component={Read} />
+            <Route path="/paste/search" component={Search} />
             <Route path="*" component={NotFound} />
           </Routes>
           </div>

frontend/src/api/client.ts

@@ -56,14 +56,15 @@ const findAll = (): Promise<Array<PasteListView>> => {
 
 const searchAll = (term: string): Promise<Array<PasteSearchView>> => {
   const params = new URLSearchParams([['term', term]]);
-  const url = new URL('/api/v1/paste/search?' + params.toString(), apiBaseUrl());
+  const url = new URL('/api/v1/paste/search?' + encodeURI(params.toString()), apiBaseUrl());
 
   return fetch(url)
     .then(value => value.json())
-    .then(value => value.pastes);
+    .then(value => value.pastes)
+    .catch(_ => [])
 }
 
-const deletePaste = (id: string) => {
+const deletePaste = (id: string): Promise<void> => {
   const url = new URL('/api/v1/paste/' + id, apiBaseUrl());
 
   return fetch(url, {

frontend/src/components/CreatePaste/CreatePaste.tsx

@@ -36,6 +36,7 @@ const CreatePaste: Component<CreatePasteProps> = ({onCreatePaste, initialPaste})
   const [lastPasteUrl, setLastPasteUrl] = createSignal<string>();
 
   let creationForm: HTMLFormElement
+  let submitInput: HTMLInputElement
 
   const updateFormField = (fieldName: keyof FormModel) => (event: Event) => {
     const inputElement = event.currentTarget as HTMLInputElement;
@@ -81,6 +82,7 @@ const CreatePaste: Component<CreatePasteProps> = ({onCreatePaste, initialPaste})
         resetStore();
         setLastPasteUrl(url);
       })
+      .catch(e => submitInput.style.backgroundColor = 'red');
   }
 
   return (
@@ -140,8 +142,8 @@ const CreatePaste: Component<CreatePasteProps> = ({onCreatePaste, initialPaste})
         <div>
           <textarea minLength="5"
                     maxLength="4096"
-                    required={true}
-                    autofocus={true}
+                    required
+                    autofocus
                     rows="20"
                     cols="75"
                     placeholder="Paste here"
@@ -153,7 +155,7 @@ const CreatePaste: Component<CreatePasteProps> = ({onCreatePaste, initialPaste})
         <Show when={lastPasteUrl()}>
           <p class={styles.lastPaste}>{lastPasteUrl()}<Copy/></p>
         </Show>
-        <input type="submit" value="Paste"/>
+        <input ref={submitInput} type="submit" value="Paste"/>
         <input type="reset" value="Reset"/>
       </fieldset>
 

frontend/src/components/CreatePaste/createPaste.module.css

@@ -65,7 +65,7 @@
 .createForm input[type=submit],
 .createForm input[type=reset] {
     display: inline-block;
-    width: initial;
+    color: var(--color-text);
     margin: .5rem;
 }
 

frontend/src/components/RecentPastes/RecentPastes.tsx

@@ -76,6 +76,8 @@ const RecentPastes: () => JSX.Element = () => {
             <span class={styles.refetch} onClick={manualRefetch}>↻</span>
           </h3>
 
+          <A class={styles.searchLink} activeClass={styles.searchLinkActive} href={'/paste/search'}>Search all pastes</A>
+
           <ol>
             <For each={pastes()}>{item =>
             <li class={styles.item}>

frontend/src/components/RecentPastes/recentPastes.module.css

@@ -14,7 +14,6 @@
     list-style: none;
     margin: 0;
     padding: 0;
-
 }
 
 .recentPastes .item {
@@ -50,3 +49,12 @@
         border-bottom: 1px dotted #a5a5a5;
     }
 }
+
+.searchLink {
+    margin-left: 1rem;
+    margin-bottom: 1rem;
+}
+
+.searchLinkActive {
+    display: none;
+}

frontend/src/components/SearchPastes/SearchPastes.tsx

@@ -1,13 +1,25 @@
-import {createResource, createSignal, JSX} from 'solid-js';
+import {Component, createResource, createSignal, JSX, Show} from 'solid-js';
+import {A} from '@solidjs/router';
 import ApiClient from '../../api/client';
-import {PasteListView} from '../../api/model/PasteListView';
+import {PasteSearchView} from '../../api/model/PasteSearchView';
+import {toDateTimeString} from '../../datetime/DateTimeUtil';
+import styles from "./searchPastes.module.css";
 
-const SearchPastes: () => JSX.Element = () => {
+type SearchPastesProps = {
+  term: String
+  pastes: PasteSearchView
+  onSearchEnter: (term: String) => void
+}
+
+const SearchPastes: Component<SearchPastesProps> = ({term, pastes, onSearchEnter}): JSX.Element => {
 
   const [search, setSearch] = createSignal<string>();
 
   const [results, { refetch }] = createResource(() => search(), () => searchTerm());
 
+  let searchInput: HTMLInputElement;
+
+
   const searchTerm = (): Promise<Array<PasteListView>> => {
     if (search() && search().length >= 3) {
       return ApiClient.searchAll(search());
@@ -21,17 +33,56 @@ const SearchPastes: () => JSX.Element = () => {
     refetch();
   }
 
-  const submitSearchForm = (e: Event) => {
+
+  const submitOnClick = (e: Event) => {
     e.preventDefault();
-    refetch();
+    if (searchInput.value?.length >= 3) {
+      onSearchEnter(searchInput.value);
+    }
+  }
+
+  const submitOnEnter = (e: Event) => {
+    e.preventDefault();
+
+    if (searchInput.value?.length >= 3) {
+      if (e instanceof KeyboardEvent && e.key === "Enter") {
+        onSearchEnter(searchInput.value);
+      }
+    }
   }
 
   return (
-    <div>
+    <>
+      <form autocomplete="off" class={styles.searchForm} onSubmit={submitOnClick}>
+        <fieldset>
+          <input ref={searchInput} onKeyUp={submitOnEnter} value={term} type="search" required minlength="3" maxlength="25" placeholder="Search for pastes" autofocus />
+          <input type="submit" value="Search"/>
+          <input type="reset" value="Reset"/>
+        </fieldset>
+      </form>
+
+      <Show when={term}>
+      <Show when={pastes.length} keyed fallback={<p>Nothing found</p>}>
+
+      <ol styles={styles.searchResults}>
+        <For each={pastes}>{item =>
+        <li class={styles.item}>
+          <p><A href={'/paste/' + item.id}>{item.title || 'Untitled' }</A></p>
+          <p>
+            Created: <time>{toDateTimeString(item.dateCreated)}</time> |
+            Expires: <time>{item.dateOfExpiry ? toDateTimeString(item.dateOfExpiry) : 'Never'}</time> |
+            Size: {item.sizeInBytes} bytes
+          </p>
+          <pre><i>“{item.highlight} [..]”</i></pre>
+        </li>
+        }
+        </For>
+      </ol>
 
-      <h1>BinPastes</h1>
+      </Show>
+      </Show>
 
-    </div>
+    </>
   )
 }