streamdeck-ui: update xlib version constraint

Description of changes
python-xlib was upgraded to 0.33 in 671ba422dd4652850a4ff2d53c8ca6c3ab3bce5e. That broke streamdeck-ui because it only accepts 0.31. This PR patches  streamdeck-ui to upgrade its python-xlib dependency. (Upstream PR: https://github.com/timothycrosley/streamdeck-ui/pull/289)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build streamdeck-ui


Update README.md

No dnf in RHEL 7.2.
Reference yum + dnf?

You rock for taking the time to offer that. I've since completely re-written the readme.


Proposed Feature: Auto-tag NSFW images using prompt tokens

Using a configurable list of trigger tokens, auto-tag images in the database as NSFW

Done


Complete the README

Could you explain in the README how to install and run ssherver for Go newbe and show an example of config.yml ?

:+1:

This is not really meant to be easy to run yourself, but it's now entirely environment variable based, so there is no config.yml.


Issue 985

Resolved / Related Issues
Items resolved / related issues by this PR.

Maybe closes #985

Details of Changes
Add details of changes here.

Bring apps opened by the fulltrustprocess to the foreground (but does not activate the window)

Validation
How did you test these changes?

[ ] Built and ran the app

@yaichenbaum I'm posting here the old code I had. Won't be able to work on it this week, so feel free to do stuff with it.

OK I have no idea if this works as I was never able to reproduce the issue in question.
Basic idea: after launching a new process, find any new windows that were open and bring them to the foreground

Can't use "process.MainWindowHandle" as it does not work for some apps (e.g windows terminal)
Can't use "SetForegroundWindow()" because it only work when called from the foreground process

@gave92 I'll merge this and we'll hopefully find out in the long run if it solved the issue.


[General] Developer Guide: Exception handling when multiple fields are invalid for Future Enhancements

Currently we don't handle properly when multiple inputs are invalid.
For example, with invalid input job and invalid tag are put in that succession, exception will return only the job exception in the feedback box, and not the invalid tag exception.
Future Enhancements can consider printing out multiple exception messages at once in the feedback box so user rectifies all invalid fields immediately/

rebuttal:

You fix the first shown field
Then fix the subsequent incorrect fields.

If asked why list one by one, then say:

So that we know which errors to fix one by one.


line height

Testing the increase/decrease line height and it would appear not to work as expected.
Increase seems ok but neither descrease nor reset will get the [age backl to the previous behaviour.
I made a codepen  of an official bootstrap example plus this plaugin so you can see for yourself

@brianteeman I can see that the reset doesn't really get it back to the same exact size (almost the same but not - we can fix that) but I cannot see why the decrease doesn't work as expected.


[Styling]: Social media handles change to their original default color

Is your feature request related to a problem? Please describe.
It's not related to a specific problem it is only about standardizing the color scheme across social media platforms to a default color, simplifying the user interface experience.
Describe the solution you'd like
Changing to default color

Alternatives
.
Additional Context
.
Are you interested in contributing?

[x] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc

Can you assign me this task under Gssoc'24?

Hey @Scrapper700, please send link to that page. I can't find it.

https://digitomize.com/
right corner..

We don't even have youtube??

I just gave that as a sample.. Mainly I am talking about the social media icons only.. Since I am new to this that is why wanted to start with this styling @pranshugupta54

We'll keep the white theme for now, you can work on other good first issues


Change the order of some built-in policies and client policies

closes https://github.com/Azure/azure-sdk-for-cpp/issues/2655
I made this change so that the HTTP headers x-ms-client-request-id and User-Agent are visible to customers. They can do something accordingly (overwrite or log) if they want.
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

are visible to customers

They are visible if they use a per-retry policy.  @JeffreyRichter is it OK/Expected to let customers access these 2 headers?  Just asking in case we might be missing something.


Blob Filtering Advice is Misleading/Wrong

This documentation around filtering on this page is highly misleading.

If you want to ensure that the Microsoft.Storage.BlobCreated event is triggered only when a Block Blob is completely committed, filter the event for the CopyBlob, PutBlob, and PutBlockList REST API calls. These API calls trigger the Microsoft.Storage.BlobCreated event only after data is fully committed to a Block Blob. To learn how to create a filter, see Filter events for Event Grid.

This language leads users to believe that PutBlob is an atomic commit for a blob, and it is not.  It is an atomic BLOCK operation and will be called many times while a blob is being uploaded.  Users should instead filter to PutBlockList.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 98043a34-9c8a-67b1-78f7-1dc322100e79
Version Independent ID: e83df230-4d9f-943c-bd71-f96210b8c469
Content: Azure Event Grid blob storage event schema
Content Source: articles/event-grid/event-schema-blob-storage.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@don4of4 Hope you have self resolved the issue and hence closed the issue. Thanks for the update.

@don4of4,
Your statement regarding PutBlob is not correct. It is a single atomic operation that creates a blob up to 256 MB in a single operation.

The event that is not emitted and not listed here is PutBlock, which is the atomic block operation used for multi-part uploads of blobs of any size (0 bytes up to ~5TB). That blob is then committed with a PutBlockList operation.

Agreed, I closed this ticket after testing the scenarios with debug logs.
There were a couple conflicting docs which have been corrected. Thank you for clarifying on this ticket.


Search by participant count

Update the API to allow searching by the participants query param.
Parameters can be combined into one filter in the query (e.g. api/activity?key=5881028&participants=1 will match the same activity as api/activity?key=5881028, but api/activity?key=5881028&participants=2 will not find anything). As a result, adding the parameter to Bored.now as a keyword argument is sensible.

Fixed by #14


Extensions for sequence comparing

What about adding extensions allowing producing "diffgrams" between sequences of objects with value and id identity?
The desired behavior would be:
        [TestMethod]
        public void Compare_Value_Identity()
        {
            var source = new[] { 1, 2 };
            var modified = new[] { 2, 3 };

            var (inserted, deleted) = source.Compare(modified);

            CollectionAssert.AreEqual(new[] { 3 }, inserted.ToArray());
            CollectionAssert.AreEqual(new[] { 1 }, deleted.ToArray());
        }

And:
        [TestMethod]
        public void Compare_Id_Identity()
        {
            var source = new[] { (1, "A"), (2, "B"), (3, "C") };
            var modified = new[] { (2, "B"), (3, "D"), (4, "E") };

            var (inserted, updated, deleted) = source.Compare(modified, x => x.Item1);

            CollectionAssert.AreEqual(new[] { (4, "E") }, inserted.ToArray());
            CollectionAssert.AreEqual(new[] { (3, "D") }, updated.ToArray());
            CollectionAssert.AreEqual(new[] { (1, "A") }, deleted.ToArray());
        }

Library code is:
    public static class SequenceChange
    {
        public static (IEnumerable<T> Inserted, IEnumerable<T> Deleted) Compare<T>(
            this IEnumerable<T> source, IEnumerable<T> modified) =>
            source.Compare(modified, EqualityComparer<T>.Default);

        public static (IEnumerable<T> Inserted, IEnumerable<T> Deleted) Compare<T>(
            this IEnumerable<T> source, IEnumerable<T> modified, IEqualityComparer<T> comparer) =>
            (modified.Except(source, comparer), source.Except(modified, comparer));

        public static (IEnumerable<T> Inserted, IEnumerable<T> Updated, IEnumerable<T> Deleted) Compare<T, TKey>(
            this IEnumerable<T> source, IEnumerable<T> modified, Func<T, TKey> keySelector) =>
            source.Compare(modified, keySelector, EqualityComparer<T>.Default);

        public static (IEnumerable<T> Inserted, IEnumerable<T> Updated, IEnumerable<T> Deleted) Compare<T, TKey>(
            this IEnumerable<T> source, IEnumerable<T> modified, Func<T, TKey> keySelector, IEqualityComparer<T> comparer) =>
            (modified.ExceptBy(source, keySelector),
            from s in source
            join m in modified on keySelector(s) equals keySelector(m)
            where !comparer.Equals(s, m)
            select m,
            source.ExceptBy(modified, keySelector));
    }

you can get this result using FullGroupJoin operator.

@dmitrynogin Thanks for your proposal but I think ListDiff by @praeclarum should be quite sufficient for this so I don't see much point/motivation in maintaining duplicate functionality separately as part of this library.


router.beforeEach with  '/:pathMatch(.)'  can not work perfectly.

Version
4.0.0-rc.3
Reproduction link
https://codepen.io/no-map/pen/ZEONNxo
Steps to reproduce
i am using router.beforeEach function to add async  routers.
as follows
router.beforeEach(async (to, from, next) => {
  if(noServiceRoutes) {
    await getServiceRoutes()
    router.addRoute()
    next({ ...to, replace: true })
  }
  ...
})

this is work.
but  when i use '/:pathMatch(.)' to redirect all notFound routes to 404
 {
    path: '/:pathMatch(.*)*',
    redirect: '/404'
  },

something wrong.
when i refresh my brower.  router.beforeEach can not work.  allways redirect to 404 pages.
What is expected?
i think router.beforeEach shoud work.
What is actually happening?
router.beforeEach did not work.

please tell me how to make it works. thanks

This is expected because redirect records trigger before all navigation guards. In general, don't do a redirect of a catch all to a fixed url, it removes the information of the route that didn't exist and is confusing to users. A workaround in your case is to use a beforeEnter: https://next.router.vuejs.org/guide/advanced/navigation-guards.html#the-full-navigation-resolution-flow

however, it worked perfectly on my another computer....i don't know what happened. so strange.

however, it worked perfectly on my another computer....i don't know what happened. so strange.

i think i found the reason. i  give up using redirect property to catch all not found.
{
    path: '/:pathMatch(.*)*',
    component: NotFound,
  },

but still don't know why this can work.

I have the same problem.When I use Addroute, the refresh browser redirects to 404.How did you solve?

I have the same problem.When I use Addroute, the refresh browser redirects to 404.How did you solve?

as follows
// router.js
const defaultRouters = [
  {
    path: '/login',
    name: 'Login',
    component: Login
  }, {
    path: '/404',
    name: 'NotFound',
    component: NotFound
  }, {
    path: '/401',
    name: 'Forbidden',
    component: Forbidden
  }, {
    path: '/',
    name: 'Layout',
    redirect: '/home',
    component: Layout,
    children: [
      {
        path: '/home',
        name: 'Home',
        component: Home,
        meta: {
          title: "Home"
        }
      }
    ]
  },
  {
    path: '/:pathMatch(.*)*',
    component: NotFound,
  },
]

addRoute function
router.addRoute('Layout', route);


how to remove shadow in mask position in the output image

how to remove shadow in mask position in the output image

such problems are usually occurred in recovering large color space, which are not significant in real-world cases. We recommand to use soft mask to solve this problem trivially (using masks with continue values from 0 to 1 in the boundary areas). soft mask here indicates blur the binary mask, can be implemented through nearst neighbourhood upsampling the mask following by bilinear downsampling the mask.
we also think that training for more iterations could alleviate this problem

Missing specific masking types during the training may also cause this problem (such as missing rectangular masks during the training).

m 0 to

such problems are usually occurred in recovering large color space, which are not significant in real-world cases. We recommand to use soft mask to solve this problem trivially (using masks with continue values from 0 to 1 in the boundary areas). soft mask here indicates blur the binary mask, can be implemented through nearst neighbourhood upsampling the mask following by bilinear downsampling the mask. we also think that training for more iterations could alleviate this problem

How to implement it in python , can you provide a demo ?

you can use cv2.resize, first set interpolation=INTER_NEAREST and upsampling the mask, following by setting interpolation=INTER_LINEAR then downsampling the mask to original resolution.

you can use cv2.resize, first set interpolation=INTER_NEAREST and upsampling the mask, following by setting interpolation=INTER_LINEAR then downsampling the mask to original resolution.

not useful

you can use cv2.resize, first set interpolation=INTER_NEAREST and upsampling the mask, following by setting interpolation=INTER_LINEAR then downsampling the mask to original resolution.

not useful

mask = cv2.resize(mask, (w*2, h*2), interpolation=cv2.INTER_NEAREST)
mask = cv2.resize(mask, (w, h), interpolation=cv2.INTER_LINEAR)

mask = cv2.resize(mask, (w2, h2), interpolation=cv2.INTER_NEAREST)
mask = cv2.resize(mask, (w, h), interpolation=cv2.INTER_LINEAR)
(not useful)
pixel inside the mask is  2 pixels( or 1 pixels) smaller than pixel outside the mask

such as missing rectangular masks during the

what does missing rectangular masks mean? only irregular and segmentation mask is not ok?

such as missing rectangular masks during the

what does missing rectangular masks mean? only irregular and segmentation mask is not ok?

That means we do not use rectangular masks during our training. We think only irregular and segmentation mask is not enough for inpainting model to generalize well to other mask types.

how to remove shadow in mask position in the output image

Hello, I also met the same problem, may I ask whether you have solved this problem


Configuration nodes - Cannot read property 'set' of undefined

I'm running node 0.12.2 and node-red 0.11.1 locally and on heroku and bluemix. My configuration node works fine locally, however on heroku and bluemix the new configuration modal fails to popup and Chrome console shows Uncaught TypeError: Cannot read property 'set' of undefined. See this short video.
Here's the offending code. It seems to be only if the code is installed from npm?

You haven't included connection-config in the declared nodes in your package.json.
We don't tend to have config nodes declared in files separate to the nodes that use them, so not hit this particular scenario before. The editor should provide a better error message.

Bingo! That did the trick!
I wasn't sure if I needed it in the package.json since it was a config node and I couldn't find any other nodes that included it.
Thanks.

As per my comment, the editor could provide a better message for this which is why the issue stayed open.


any workaround in case of returning null in mysql with knex

#Hi,
I use join-monster with mysql dialect (RDS aurora) and I finally got the problem mentioned at
http://join-monster.readthedocs.io/en/latest/call-function/
If you are using Knex, be careful. The raw method will return different objects depending on the knex dialect being used. The exact snippet above may not work with MySQL, MariaDB, etc.
I saw this in the following case at first:
https://github.com/stems/join-monster/issues/167
SQL generated by joinmoster works great in SQLWorkbenchJ, just a simple SQL client, but it returns
null in knex.raw().
I am not sure what do I need to do for solving this problem. Any workaround or where do I need to start to solve it?
p.s, I posted similar question at knex issue tracker.

I am going to close this because I don't know how to do correctly. Per one of knex maintainer, it is not related to the project. So, I am going to use mysql driver in this case.


Grub boot time optimizations

grup: use quiet boot

Nit: grup -> grub
Only for RPI? Do the same for RPI or wait with it for now? Would also be interesting to know approximately how much time is saved.

Only for RPI? Do the same for RPI or wait with it for now?

Could you please elaborate that?

Would also be interesting to know approximately how much time is saved.

If there are no errors printed on the console (#370 fixes one), ~1500 ms.

Only for RPI? Do the same for RPI or wait with it for now?

Could you please elaborate that?

This is only for Intel/ARP but we want to add quiet for RPI as well?

Would also be interesting to know approximately how much time is saved.

If there are no errors printed on the console (#370 fixes one), ~1500 ms.

Nice.

Still should change "grup:" to "grub:" in second commit.
And... I guess you want to punt adding quiet for RPI... ?
And you have not answered if "ESC" worked or not if the menu is completely hidden. I am fine with punting this if we track it somewhere (assuming this is behavior we want... I think it is... hide grub completely, show splash as soon as possible... and make it so the grub menu can be brought up if really wanted).


Wallet

Pls money was sent using my old wallet address that was assign for me a month ago by copay. Will it still enter my account ? Pls help

Hello @1234aeda
To protect your privacy, the BitPay wallet generates new addresses automatically once you use a bitcoin address once. Wallet addresses do not expire – any payments sent to past addresses will continue to appear in your balance.
Please see this article for more information: Why does my BitPay wallet's bitcoin address keep changing?
Kind regards.

we have developed our own hardware for cracking all versions of bitcoin wallet password with superfast speed
and if you still has problem I can help you run it. you can find my contact details on my profile
or you can send me message in telegram: @viladmir2022 or gmail<EMAIL_ADDRESS>


adds support for printing the RDD debug string

This mimics the pySpark toDebugString method. Useful for diagnosing how spark is splitting a pipeline up into stages and also for inspecting the parallelization (number of partitions) at each stage.

@mattpollock this is a great addition! One suggestion, it currently prints like:

But it would be nicer to print with cat inside spark_debug_string, with something that would look like:

I can make this change if you are running short on time, let me know.

Yeah, I agree. I've been using cat() locally. Do you think there might be a use case where someone wants to capture the output to a character vector and programatically do something like determine the choke point (stage with min number of partitions)? That sort of thing will be more annoying with cat.

Hmmm, this an #1519 are failing in travis for Spark 2.3 on the exact same error
/home/travis/.travis/job_stages: eval: line 57: syntax error near unexpected token `('
/home/travis/.travis/job_stages: eval: line 57: `Rscript -e covr::codecov() '

despite my branching off master and making completely independent changes for both PRs. Is something in the CI job busted or am I doing something really stupid here?

@mattpollock we've seen some intermittent issues in Travis, I'll just rerun the job to get a clean pass before merging this one.

@mattpollock renamed spark_debug_string() to sdf_debug_string(), all our Spark DataFrame operations are prefixed sdf_, see https://github.com/rstudio/sparklyr/commit/e679cb129afe89d8b3b65b281ef8cdb396abc2d1.

Thanks!


Add web version for Firefox

For https://github.com/fregante/webext-alert/issues/1
This is the web-only 2021 version of webext-alert
To do

[x] Publish to Vercel
[x] Set it as a fallback for Firefox

MV2 test successful!


Custom AuthorizationMessageHandler not working

I have been following this documentation: https://docs.microsoft.com/en-us/aspnet/core/security/blazor/webassembly/hosted-with-azure-active-directory?view=aspnetcore-3.1
However my client and server are not hosted at the same url. Therefore I came to this page and followed the instructions about creating a custom AuthorizationMessageHandler.
The following works:

Accessing a component -> Redirected to azure ad login and back to app
In the browser I can see the token etc. in the session storage
Calling an unsecured server api method and getting data

But as soon as I add the code about the custom AuthorizationMessageHandler the same call to the server api method throws and runs into an infinite loop with the try/catch -> AccessTokenNotAvailableException.Redirect().
I have no idea how to fix this or even where to look at... any help please?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f2cdb52c-843f-5944-429d-11ea821c1e22
Version Independent ID: 9c4f99ef-f070-b527-4efc-96a04798d9ef
Content: ASP.NET Core Blazor WebAssembly additional security scenarios
Content Source: aspnetcore/security/blazor/webassembly/additional-scenarios.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @guardrex
Microsoft Alias: riande

Ok I was able to get it to work. I was missing the scope information.
Feedback: The documentation already has a hint about adding scopes but to me it was not to clear what I have to put. I tried some values but they all resulted in authentication failures - but it worked when leaving it away so that's what I did. The token I got was even valid for the server when sending it over curl or some other test tool BUT then the AuthorizationMessageHandler didn't work. I now assume this is because of the IAccessTokenProvider that is internally used will not find the item in the browser storage (it is named "msal.SCOPE.idtoken") if the scope is not specified... a meaningful exception message would make this clear and prevent that people like me need to spend hours to figure it out!

Hello @srudin ... Thanks for that feedback.

a hint about adding scopes

A scope is required for an API call, and I think you're the second dev to ask on that point. This topic will be worked on further soon, and I'll try to make that point clearer in all of the Blazor WASM security topics when I make the next round of updates.

it was not to clear what I have to put

I don't think that we'll be able to address that generally. That's variable depending on the provider. It can even be different between v1.0 and v2.0 endpoints.

will not find the item in the browser storage (it is named "msal.SCOPE.idtoken") if the scope is not specified... a meaningful exception message would make this clear

We only work on the docs here. You would need to address that to engineering in an issue on their repo at ...
https://github.com/dotnet/aspnetcore/issues
If you open an issue with them, please add a cc: @guardrex to your opening comment so that I can follow the discussion.


Unwanted input element on left-top most corner

Actual Behaviour
On the left most corner, there is irritating input element, that seems of no use.
Expected Behaviour
Either it should be ommited or it should be used with visibility none..
Steps to reproduce it
Just visit P5js.org
screenshot

Would you like to work on the issue?
Yes I would love to solve this issue

Sorry that the screenshot i uploaded is rotated😅

Thanks. This is covered by #559, I'm going to close this to track everything related there.


Alert if it takes a long time to create realm

This PR creates a sentry alert if creating a realm takes too long

Do we feel confident that we are already notified in the case that realm creation never completes?

yeah in those cases we have seen additional exceptions (up to now those scenarios were where the matrix registration shared secret was not specified property


RATIS-1051. Use try-resource-block when saveMd5File

What changes were proposed in this pull request?
Use try-resource-block when saveMd5File
What is the link to the Apache JIRA
RATIS-1051
How was this patch tested?
No need.

@runzhiwang Please take a look at this PR. no try-resource-block cannot ensure the afos will close when exception happen.

+1
@maobaolong Thanks for the patch, I have merged it.


Where should I set the "azkaban.native.lib"

I have tried several places to populate the "azkaban.native.lib" on Azkaban 3.13.0, but the Job never run up successfully.
Please help to advice.

azkaban.utils.UndefinedPropertyException: Missing required property 'azkaban.native.lib'
at azkaban.utils.Props.getString(Props.java:478)
at azkaban.jobExecutor.ProcessJob.run(ProcessJob.java:120)
at azkaban.execapp.JobRunner.runJob(JobRunner.java:693)
at azkaban.execapp.JobRunner.run(JobRunner.java:545)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
19-01-2017 18:10:54 CST SparkPi ERROR - Missing required property 'azkaban.native.lib' cause: null

Kind Regards,

I found the path: azkaban_executor/plugins/jobtypes

How would you launch the solo server process, through Intellij/Eclipse?
You would better run
ligradle installDist
under azkaban-solo-server folder.
Then run it from azkaban-solo-server/build/install/azkaban-solo-server


"Uncaught TypeError: node.removeEventListener is not a function" when use Svelte with jQuery on the same page

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script>
<script src="../dist/main.js"></script>

When both Svelte and jQuery are included on the page there is an exception "Uncaught TypeError: node.removeEventListener is not a function"
The issue is caused by this function that is generated in the global namespace and rewrites a method of a window object
function removeEventListener ( node, event, handler ) {
	node.removeEventListener ( event, handler, false );
}

Exception is thrown when jQuery calls this function
/**
 * The ready event handler and self cleanup method
 */
function completed() {
	document.removeEventListener( "DOMContentLoaded", completed, false );
	window.removeEventListener( "load", completed, false );
	jQuery.ready();
}

In this case the compiled function is called but not native method and the first argument is "load" instead of DOM node.
The advised solution is to wrap all code generated by the compiler to a self invoked function and avoid rewriting of window.removeEventListener() method.

This issue might be related to https://github.com/rollup/rollup-plugin-svelte , so I opened it there and closed here


How can I add an SSL cert to my monitoring?

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hi, How can I use let's encrypt to run an SSL cert for our monitoring system? at the moment it is warning constantly that it's not secure but we want it to have an SSL as client will be visiting it.
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.13
💻 Operating System and Arch
Azure Ubuntu 22.04
🌐 Browser
Edge 126.0.2592.87 (Official build) (64-bit)
🖥 Deployment Environment

Runtime: 24.0.7
Database:
Filesystem used to store the database on: linux filesystem
number of monitors: 15

found a way around it using the tunnel/


Max length for memo and payee_name does not match what YNAB allows

Hi there! 👋🏻
First I've got to state again how happy I am using YNAB and the API for my own workflow, this is working great!
I recently noticed that some field maximum lengths might not match the actual length limits when using the YNAB web client.
In short, the OpenAPI spec file specifies a maxLength for payee_name and memo fields of respectively 50 and 200, but it's been possible for a long time to create transactions with longer payee name and memo. (I just created a transaction with a 10,000-character long memo!)
I think this issue is very similar to this other issue I opened back in February last year: Validation error on API response for TransactionDetail#import_payee_name, and that it could get a similar fix (Remove unnecessary maxLength validation for import payee fields).
One thing I noticed that might have not been the case last year, is that even if I remove the validations Ruby-side, the API actually responds with an error too:
{"error":{"id":"400","name":"bad_request","detail":"memo is too long (maximum is 200 characters) (index: 19), memo is too long (maximum is 200 characters) (index: 36), memo is too long (maximum is 200 characters) (index: 37)"}}

I could imagine this is happening if YNAB's API server code is also generated with the OpenAPI Generator, based on the same OpenAPI spec. In this case, the fix would require not only a new version of this gem but also an update of YNAB's server-side API code.
Hope this makes sense, but feel free to let me know if you need more details! 🙏🏻

@davidstosik thanks for the feedback!
The API does indeed have smaller limits for some fields; this is intentional and by design.  Having said that, if you have a specific use case for increasing these a bit, we can consider that.

@bradymholt thanks for the reply!
As for a use case, you can check my previous issue that I linked in the description above: basically, it's impossible to write a script that fetches existing transactions, changes them then saves them again, since you might be fetching transactions that were created in the UI and will appear invalid from the API.

Ah, I see.  Thanks for the context.  As I mentioned, we intentionally limit these in the API to prevent abuse.  Is there a limit we could increase it to (short of the limit the apps allows) that would allow you to proceed, even if you needed to trim in some cases when saving a transaction from the API?

@bradymholt Sorry it took me some time to get back to you.
I took a look at my whole budget:
# frozen_string_literal: true

require "ynab"

puts "Fetching data..."

ynab_api = YNAB::API.new(ENV["YNAB_ACCESS_TOKEN"])
budget = ynab_api.budgets.get_budgets.data.budgets.sort_by(&:last_modified_on).last
transactions = ynab_api.transactions.get_transactions(budget.id).data.transactions

puts <<~OUTPUT
  Fetched #{transactions.count} transactions.
  Three longest memo sizes: #{transactions.filter_map(&:memo).map(&:length).uniq.max(3).join(", ")}
  Three longest payee name sizes: #{transactions.filter_map(&:payee_name).map(&:length).uniq.max(3).join(", ")}
OUTPUT

Fetching data...
Fetched 5225 transactions.
Three longest memo sizes: 273, 200, 199
Three longest payee name sizes: 100, 98, 92

So I've got old transactions in my budget with memo and payee name that exceed the current validations. (I'm actually surprised the YNAB gem did not fail to load the data, as it does with invalid flag colors, here. 😅)
I'm writing an import script that allows me to import into YNAB transactions from my various Japanese bank account. (There's no account linking here!)
On of the specificities is using a separate account for Amazon transactions, where I'd like to use the product name as memo.
That's where I hit a wall, as some product names can exceed the 200-character limit easily.
The payee name is less of a problem I guess.
I understand that removing the limits would favor abuse, but do you think that it would be possible to:

augment the current limits for memo and payee name to respectively 500 and 200 characters?
maybe add those limitations to the UI as well, preventing discrepancy between UI and API? (Currently, how would I save a transaction that I fetched using the API then modified locally, if it has an invalid memo?)

@davidstosik - Thanks for the feedback.  I understand the friction that can be caused by having different limits on the clients and the API.  I'll raise this internally to see about some better alignment there.  In the meantime, we can increase the limits on allowed by the server for memo and payee_name` to 500 and 200, respectively.  We'll work on a server change to get this out.

Update - This increase in limit is scheduled to ship early next week.

@bradymholt thank you very much!

@bradymholt May I ask for an update on this? (Nothing urgent, just wondering if anything went out, since I don't see a new release of the gem.) 🙏🏻

@davidstosik - The server change (1.72.1) went out back in July but we didn't mention on the release notes since it was a very minor patch.  Honestly, I didn't think a new client would be needed for this change but now I see that indeed there is a some validation logic that checks the length against the spec.  I'll cut a new release here in just a minute!  Thanks for the ping.


RULE-SET,https://raw.githubusercontent.com/Ja7ad/ShadowRocket/main/rules/proxy/whatsapp.list,PROXY

Config


Crash bug fix + optimizations in LoadObjWithCallback()

Move heap-based objects like vectors and strings outside the parse loop in LoadObjWithCallback().

This avoids reallocation on every use making the code faster.

Fix a crash bug in LoadObjWithCallback() when passing a nullptr as MaterialReader *readMatFn.

This preserves the existing API. If nullptr is passed then the material file will be ignored. This is useful when the user is not interested in the materials but only in the geometry.
Note that if nullptr is not a valid option for this function, then the API needs to be changed to take MaterialReader&.

Cool! Thanks!


please change project title description to ...

GPS calibration with point cloud by SLAM  改为 A meter level precision GPS calibrator for out-door robots by fusion sensors of GPS satellite, point cloud, IMU and others. SLAM, Gaussian process and Kalman filter algorithm is used.

ok. That's done.


Documentation: Tutorials 1 and 2 are out of date

The two tutorials call for the use of 'package:flame/game.dart' which Flutter 2.2.0 refuses to compile saying  'The library is legacy, and should not be imported into a null safe library.

Unsure why you are seeing that issue. Both the tutorials and flame already uses null safe code, just to be sure I tested the code and it worked fine on flutter 2.2.0.
I wonder if you are trying to use flame 0.29.4? The tutorials are bound to flame v1, the latest being 1.0.0-releasecandidate.11

Exactly the reason, I’ using 0.29.4 as the latest version from pub.dev. Where would I get Version 1 from please
John

On 25 May 2021, at 15:43, Erick @.***> wrote:
Unsure why you are seeing that issue. Both the tutorials and flame already uses null safe code, just to be sure I tested the code and it worked fine on flutter 2.2.0.
I wonder if you are trying to use flame 0.29.4? The tutorials are bound to flame v1, the latest being 1.0.0-releasecandidate.11
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flame-engine/flame/issues/821#issuecomment-847929774, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABMRMZPM6KF3RVRKQCFLHNDTPOZSVANCNFSM45POWRDA.

It is on pub as well, it is listed as pre release as it hasn't been fully released yet.
https://pub.dev/packages/flame/versions/1.0.0-releasecandidate.11
Closing this as this isn't really an issue on Flame, but feel free to add addition comments if there is still doubts


Babble chat alert always showing 11 alerts. Even all have been read successfully.

@gdpelican  Even after successful reading of chat messages. discourse babble plugin keeps showing alert. Same issue on all browsers and even mobile app. see following screenshots for detail.
Discourse version: v2.2.0.beta4 +11
Babble version: 4.0.7

Hey @mfarooq-seladigital is this only happening for DMs, or are @mentions in the chat channels sticking for you as well?

@gdpelican i am having both cases (DMs and @mentions) in current chat for which its showing counter 8.

done


Update Resources.md

Standardization edits. Let me know if you have questions.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/Resources.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Hassio panel split

Build 2 panels.
I did not include the result of the hassio-build script as that PR is still pending.
(I have not tested this)
Related PRs:

https://github.com/home-assistant/home-assistant/pull/10589
https://github.com/home-assistant/home-assistant-polymer/pull/630
https://github.com/home-assistant/hassio-build/pull/47

I've added the /panel route back for backwards compatibility.


No option to use microphone

Hi I just recently installed this app and I noticed that when typing in the code editor there is no option to use the microphone to help write the code in the editor. I was recently using dcoder and I really enjoyed that function on it. If there's any way to include that on this app, I would be grateful. This is a great app thank you so much for building it.

Can you please clear your concern ?

Try:
Settings -> App Settings -> Keyboard mode -> SHOW SUGGESTIONS


Setting for length of history

Is your feature request related to a problem? Please describe.
Not really an issue however sometimes I want to establish a trend of where is all of X resource going when I'm running low. I can search the history to see where that resource is being delivered however that history often only goes back 3-4 supply runs for the resource. I have a very busy LTN network so there are lots of deliveries and I can't easily establish long lasting trends.
Describe the solution you'd like
It would be really nice if there was a mod setting along the lines of "number of deliveries saved in history" this setting could be set to whatever the user wants 100, 500, 5000 etc.
Additional context
https://gyazo.com/7c4c5168b7cefb6dc6d78d0ce3a55c67

Sure, I'll add this in v0.4.0. However, increasing the length of the history will degrade performance on all fronts, so keep that in mind.


lnwallet/chancloser: properly compute initial fee of cop close txn

In this commit, we modify the way we compute the starting ideal fee for the co-op close transaction. Before this commit, channel.CalcFee was used, which'll compute the fee based on the commitment transaction itself, rathern than the co-op close transaction. As the co-op close transaction is potentailly bigger (two P2TR outputs) than the commitment transaction, this can cause us to under estimate the fee, which can result in the fee rate being too low to propagate.
To remedy this, we now compute a fee estimate from scratch, based on the delivery fees of the two parties.
Fixes #6953

The current solution has a gap in it: one of the co-op close outputs might be trimmed if they are dust. However, we can only compute if an output is dust after we compute our desired starting fee (the initiator's output might be trimmed). Therefore we have a circular dependnacy here: we can only know if an output is dust after we know the fee, but we can only compute the fee accurately if we know if an output is dust.
At a glance, we can either live with the current solution (will over estimate by a bit if one of the outputs is dust), or just worry about the remote party's output (they don't pay the fee).

Marking this as ready for review, worst case we end up over paying slightly when one of the parties doesn't have an output (it was dusted).

Seeing this in the itests, seems unrelated, but investigating:
        	Error:      	Received unexpected error:
        	            	expected confirmed balance 16668165, got 16667565
        	Test:       	TestLightningNetworkDaemon/tranche03/92-of-110/btcd/wallet_import_account

Seeing this in the itests, seems unrelated, but investigating:
        	Error:      	Received unexpected error:
        	            	expected confirmed balance 16668165, got 16667565
        	Test:       	TestLightningNetworkDaemon/tranche03/92-of-110/btcd/wallet_import_account

No, we're actually coop-closing channels in that test. And it looks like the fee needs to be increased by 600 sats to 9650 here: https://github.com/lightningnetwork/lnd/blob/master/lntest/itest/lnd_wallet_import_test.go#L564
I guess we should start using appropriate fee estimators in that test instead of hard coding things. But some times hard coded fees can help detect changes in the fee estimators too, which might be useful as well in certain cases...

But some times hard coded fees can help detect changes in the fee estimators too, which might be useful as well in certain cases...

Yep indeed, in this case I think I need to add some additional logic to ignore the possibly dusted output of the remote party (since we know that ahead of time).


TypeError: Cannot read properties of undefined (reading 'split')

Hi,
Using this tool to scan for V1 CDK stacks. Works fine on most of my accounts, however on one of the accounts I am getting the following error:
TypeError: Cannot read properties of undefined (reading 'split')
    at findV1Stacks (.npm/_npx/1a70b0cb204f1c57/node_modules/awscdk-v1-stack-finder/lib/index.js:149:85)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async processListStacksResponse (.npm/_npx/1a70b0cb204f1c57/node_modules/awscdk-v1-stack-finder/lib/index.js:122:5)
    at async find (.npm/_npx/1a70b0cb204f1c57/node_modules/awscdk-v1-stack-finder/lib/index.js:94:9).

It's going wrong in the following function:
async function findV1Stacks(response, cloudformation, region, stacks) {
    var _a;
    for (const stack of (_a = response.StackSummaries) !== null && _a !== void 0 ? _a : []) {
        const getTemplateResponse = await cloudformation.getTemplate({
            StackName: stack.StackName,
        }).promise().catch((error) => console.error(`${region}: Failed to get template for stack: ${stack.StackName}. Error: ${error}`));
        var body;
        try {
            body = JSON.parse(getTemplateResponse.TemplateBody);
        }
        catch (err) {
            body = YAML.parse(getTemplateResponse.TemplateBody);
        }
        if (body.Resources.CDKMetadata) {
            const buf = Buffer.from(body.Resources.CDKMetadata.Properties.Analytics.split(':').splice(2)[0], 'base64');
            const analyticsString = zlib.gunzipSync(buf).toString();
            const majorVersion = analyticsString.slice(0, 1);
            if (majorVersion === '1') {
                stacks.push(`name: ${stack.StackName} | id: ${stack.StackId}`);
            }
        }
    }
}

Something is causing body.Resources.CDKMetadata.Properties.Analytics to be undefined.

I am assuming a role with all permissions and no explicit denies using the aws sts assume-role api.
I am using the env variable to set the credentials after.

export AWS_ACCESS_KEY_ID=...
export AWS_SECRET_ACCESS_KEY=...
export AWS_SESSION_TOKEN=...

I verified the account has CDKToolkit stacks.
Node.js version v16.14.2, npm version 8.5.0

Can you share the templates of the CloudFormation stacks in that account? Or just the CDKMetadata resource snippets from the templates?
In the meanwhile, I can add better error handling/messaging for this scenario.

Thanks for the quick response @TimQuist! This should be enough information to address the issue. Your guard-duty-master-stack was created with cdk version 1.82.0, and the v1-stack-finder does not account for how the CDKMetadata was formatted in versions that far back.

@madeline-k
Great!
Some context:
I figured out that we did in fact not roll out to this account during our deploys. Since it only contains guard-duty and pretty much never changes this went unnoticed. In our CDK projects the versions were upgraded multiple times already, but just never rolled out to this account. This is why it was still on a pretty old version. I am glad to have noticed this so now we can include this account in the deploys 😄


rancher export/rancher up round trip doesn't work with secrets

Rancher versions:
rancher/server: v1.6.5
rancher/agent: N/A
rancher/CLI: v0.6.2
Infrastructure Stack versions:
healthcheck: N/A
ipsec: N/A
network-services: N/A
scheduler: N/A
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
17.03.x-ce
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
PRETTY_NAME="Debian GNU/Linux 8 (jessie)"
NAME="Debian GNU/Linux"
VERSION_ID="8"
VERSION="8 (jessie)"
ID=debian
HOME_URL="http://www.debian.org/"
SUPPORT_URL="http://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

3.10.0-229.14.1.el7.x86_64
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Custom
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node with Bind Mounted Database
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle (with Secrets)
Steps to Reproduce:
rancher export zaim-backend
cd zaim-backend
sed -i "s|    image: dockerregistry1.infra.idef:5000/zaim_sample_processor:0.1.\d*|    image: dockerregistry1.infra.idef:5000/zaim_sample_processor:0.1.68|" docker-compose.yml
sed -i "s|      mode: '444'|      mode: 444|" docker-compose.yml # This fixes issue 9094
rancher up -u

See configs here: https://github.com/rancher/rancher/issues/9094
Results:
INFO[0000] [webserver]: Creating
INFO[0000] [cronjobs]: Creating
INFO[0000] [processor]: Creating
INFO[0000] [webserver]: Created
INFO[0000] [processor]: Created
INFO[0000] [cronjobs]: Created
INFO[0000] [webserver]: Starting
INFO[0000] [cronjobs]: Starting
INFO[0000] [processor]: Starting
ERRO[0000] Failed Starting webserver : Failed to find secret
ERRO[0000] Failed Starting cronjobs : Failed to find secret
ERRO[0000] Failed to start: cronjobs : Failed to find secret
ERRO[0000] Failed Starting processor : Failed to find secret
ERRO[0000] Failed to start: processor : Failed to find secret
ERRO[0000] Failed to start: webserver : Failed to find secret
FATA[0000] Failed to find secret

Export for secrets doesn't work correctly at the moment, so I think this is a duplicate of #9059.


Update to most recent Selenium version

The latest browser versions are often fully supported only in the latest Selenium versions.
The Selenium version currently used in jFunk is over one year old and therefore has limited browser support.
jFunk should be updated to use the most recent Selenium version and the version of its transitive dependencies.

This is fixed in jFunk 3.2.1


Reduce the size by removing unneeded fonts, images, etc

4mb for a dialog is way too much. You have added into library the whole Montserrat font family, still have the ic_luncher mipmaps, and plenty other stuff which is definitively not needed to use this library.

please udpate to v1.1.0

Just checked the branch. Mipmap folders are still there (and AFAIK libraries don't need a launcher icon), so is the whole font family which I doubt is entirely used. So is the <string name="app_name">No Internet Dialog</string> in Strings which is not needed at all, so is the application entry in the Manifest.xml which isn't needed for a library.
Please, reopen this issue until you clean all that stuff.

This files was removed from bintray repo, and from this github repo they are also removed. Check it again, please.


shareSingle Facebook & Twitter on iOS not working

Steps to reproduce
      const shareOptions = {
        url,
        social: Share.Social[social]
      }; // url is base64 data of png and social is 'FACEBOOK', 'TWITTER'

      const res = await Share.shareSingle(shareOptions);

Expected behaviour
It should open the Facebook and Twitter
Actual behaviour
App crashes
Environment

React Native version: 0.60.5
React Native platform + platform version: iOS 12.3.1

react-native-share
Version: 2.0.0

does it happens when running with example?

@jgcmarins
I didn't check the example project.
This issue happens in my react native project

share a strring of type SupportedSocialApps not Social
I  have a setup like this and seems all G
import Share, { SupportedSocialApps } from "react-native-share";
const socialType: SupportedSocialApps = "instagram";
...

const ShareOptions = {
        social: socialType
        message: shareMsg,
        url: baseImg,
        type: "image",
 };
const shareResult = await Share.shareSingle(ShareOptions);


AudioNode disconnect methods

Fixes #471
As you can see we get quite a few methods extra, this is conform https://webaudio.github.io/web-audio-api/#AudioNode . Unlike connect, disconnect does not have optional arguments (instead, we treat a missing value as wildcard when disconnecting). I'm up for debate for new names though, or possibly exposing the Option arguments...
We now bookkeep connection on the control side, and throw when one tries to disconnect a non-existent connection.
Bonus fix: disconnect without args would previously clear all connections of a node, but the spec mandates that only outgoing connections are cleared!
The node-lib needs some work after merging this to support stuff like node.disconnect(number)

Cool, really nice!

The node-lib needs some work after merging this to support stuff like node.disconnect(number)

Yup, but this should be rather straightforward (I'm starting to be quite fluent in parsing IDL now :)
For the naming what about?
node.connect(dest)
node.connect_dest_from_output_to_input(dest, output, input)
// or (...I think I prefer the first one for the symmetry with disconnect,
// but it is more consistent according to `connect`)
node.connect_from_output_to_input(dest, output, input)

node.disconnect()
node.disconnect_output(output)
node.disconnect_dest_from_output(dest, output)
node.disconnect_dest_from_output_to_input(dest, output, input)

This is a bit more verbose, but I find naming like disconnect_dest_output a bit confusing as output seems to refer to dest

Thanks for the suggestions, I applied them


[5.3] Update doc to PHPDoc

This is intended, config files and routes share the same style of docblocks.


[How To] Create or update profile

https://github.com/ASKnetCommunity/LEAD/wiki/Create-profile-from-issues-(Maintainers)

this covers the simple Beginner Version via Issue:
https://github.com/ASKnetCommunity/LEAD/wiki/Submit-New-Profile

Maybe this can be extracted/linked to as "profile image reference": https://github.com/ASKnetCommunity/LEAD/wiki/Checklist-for-a-good-profile-photo

Maintainers: This is a bit crazy but with potential for later → I like it as backup but optional, first a simpler version would be nice for Maintainers (and Learners):
https://github.com/ASKnetCommunity/LEAD/wiki/Create-profile-from-issues

this is also relevant as changes → we call them "Profile-Updates" in the labels: https://github.com/ASKnetCommunity/LEAD/wiki/Submit-profile-changes
but I think it needs a little rewrite or can be even merged with the "new" profile via issue?


Fail to build on GNU/Hurd

The first easy step was to define missing macros IOV_MAX, PATH_MAX & MAXPATHLEN.
Patch that dynamic alloc buffer instead of harcode PATH_MAX is used in the Debian package (http://anonscm.debian.org/cgit/collab-maint/memcached.git/tree/debian/patches/03_fix_ftbfs4hurd.patch, all credits to Svante Signell).
For MAXPATHLEN, i haven't touched the code, just define it if undefined.
The second step was about using UNIX domain sockets on GNU/Hurd.
The transport was hardcoded to tcp in drive_machine() when calling dispatch_conn_new() and getpeername(2) was always called even if transport is not tcp. After using c->transport, there is still a problem : t/stats-conn.t verify that the socket name appears in "stats conn" output, which is not the case at least on GNU/Hurd (and *BSD, according to docs) but only "<AF 1>". conn_to_str() calls getsockname(2) which always return an empty string for UNIX domain sockets.

Well I have no idea how to validate this, but the changes look correct and the tests pass. It's merged into the next branch and will go out in the next release.
thanks!


disable 'Rebuilding' message when notify is false.

thank you, will implement this in a slight different way and release


[ci] Add auto-updating last-successful branch

This adds a script that runs on a cron to discover the last commit where CI all passed (every job was successful and tvm-ci/branch is included) and updates a git branch last-successful that tracks this commit from main. This can be used for checking out the latest unbroken TVM, which can be useful for developers wanting a good changeset to base their changes on or for infra needing a clean, up-to-date TVM.
Also soliciting better names than green

I suggest last-successful as a tag so we're not coupled to the colour :smile_cat:

one problem here is that a tag doesn't automatically update with git fetch. A branch would. it's a little confusing to have a branch instead of a tag, but a branch is confined to origin/ namespace and will auto-update with git fetch. i think we should switch to updating a branch to avoid confusing behavior where last-successful is not updated after a fetch.
cc @leandron @mbrookhart if there are other ideas

Sorry about the overactive bot everyone, it's still missing some checks to look at before sending out these pings

Let's get this in so we can start iterating if we need to :smile_cat:

What happened to this branch? The CI job got manually disabled about 5 months ago, it seems.

I filed an issue with the reason for disabling and potential fix: #14618


Accelerator documentation for v0.14.0 release

Overview/Summary
Accelerator documentation for v0.14.0 release
This PR fixes/adds/changes/removes

Adds accelerator documentation with fixes from issue #504
#AB26310

Breaking Changes

None as documentation for new feature

Testing Evidence
Not applicable
As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[x] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[x] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[x] If relevant, created or updated Code Tours here

Please don't merge until v0.14.0 is released as it as dependencies on the release version.


Make this a Rust project once again

Subfolders were not ignored.
Docs: https://github.com/github/linguist/blob/0c7f82f88a2fb482db3a02812cbb4b8aab7308d1/docs/overrides.md#vendored-code

🎉 🎉


Project description Mode of operation issue

In project description The mode of operation truth table :
ui_in[0]	ui_in[1]    ui_in[2]
0	0	 0		 MSB Onboard Display in C
0	1	 0	         LSB Onboard Display in C
0	0	 1	         MSB Onboard Display in F
0	1	 1              LSB Onboard Display in F
1      X      0              External 7 segemnt in C
1      X      1              External 7 segemnt in F

Fixed. Thanks.


1.8.x

[x] You have read the Spring Data contribution guidelines.
[x] There is a ticket in the bug tracker for the project in our JIRA.
[x] You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

Thanks for the PR, but Spring Data GemFire 1.8.x is EOL.  Currently, only SDG 1.9.x and 2.0.x  are the actively maintained production releases.


pdfminer does not always produce clean textual output

Summary
pdfminer sometimes omits characters in the textual output.
There are some characters missing.
Environment

Reference commit/version: a7b4b0c5cce05ae8b029f121a880fc5ed2c318f1

The issue has been traced back and found to be inherent to pdfminer's current implementation of the pdf text extraction method, and somewhat on the PDF format itself.
See Issue https://github.com/aarohijohal/pdfminer.six/issues/1
Pointing to glyphs inside the font spec while the fontspec does not have a glyph->character mapping makes sure that a resultant text is not always found.
This needs to be investigated.


feat(QUI-110): initial faucet implementation

feat(QUI-110): initial faucet implementation
feat(faucet): use rotating faucet outputs to allow next-block latency

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#59  👈 (View in Graphite)
#58
#57
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Version bump v2.2.0

I'd like to release a new version of this package with the latest changes. Since there is a new feature I bumped the minor version. To generate this commit I ran:
attribution tag 2.2.0

I'd like to get the contributor guide in, and drop support for 3.7 while at it so we can bump black and flake8 to latest versions. I will try to do that tonight or tomorrow and then look at a version bump by mid week at the latest.

I will try to do that tonight or tomorrow and then look at a version bump by mid week at the latest.

Sounds good to me! Thanks for helping push a release through, will be super helpful to get the last fixes out for my use case

2.2.0 is now available on PyPI


Auth appended to the routing, no_auth + database_auth injected

Description
Auth appended to the routing, no_auth injected, database_auth partially injected.
Request for the review as proposed to just verify if the approach is correct with what was agreed
closes #5759
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Code is clear and sufficiently documented
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

@bartv regarding outlet - https://reactrouter.com/en/main/components/outlet

@bartv regarding outlet - https://reactrouter.com/en/main/components/outlet

that is a weird name :-)

I had to remove email, first name & last name input part from the keycloak scenario to make it working

Okay, still would like the OSS tests later if you have the time :)

oss-tests results

Pull request rejected by merge tool. No change entry file found

Processing this pull request

Merged into branches master, iso7 in 36a46bb60a0df9828161323b7ec91f74230320b5


Can't subscribe to newsletter

I tried to subscribe to Overthought both from your unbelievably good new site (using the Subscribe button at the top of the articles list) and directly from your Twitter profile and I get this error message preventing me from confirming my subscription when I click the link sent in Revues confirmation email (in both cases):

Oof, thanks for the heads up — it might be that you're already subscribed? If you wanna DM me your email or share it in a comment here, I can double check.

True, I used to be subscribed through Mailchimp but had unsubscribed at one point iirc.
I checked my email history and the last mail I received from you is titled "How to give a great product design portfolio presentation" from the 7th of May. Since you've published new writing since, it seemed like I was unsubscribed. I can't say for sure if I was or was not subscribed before.
Anyways, I used ye olde task specific gmail address (+) trick to subscribe with a different address and it worked. So this seems to be something that's just related to my specific account and not a general problem. I'll close this, sorry for the sorta false alarm!

Thanks for digging into it — if you get double emails on the next newsletter, we'll know you were already subscribed 😛


Flutter gallery does not scroll at 60fps on a Galaxy Note 5

Device is in our test rack.
[✓] Flutter (on Mac OS X 10.12.5 16F73, locale en-US, channel alpha)
    • Flutter at /src/flutter
    • Framework revision d36e2f6191 (4 weeks ago), 2017-06-21 15:09:10 -0700
    • Engine revision b0dee695ec
    • Tools Dart version 1.24.0-dev.6.7

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.0)
    • Android SDK at /Users/eseidel/Library/Android/sdk
    • Platform android-26, build-tools 26.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.2, Build version 8E2002
    • ios-deploy 1.9.1
    • CocoaPods version 1.2.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 3.2
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.1.4)
    • Flutter plugin version 14.0
    • Dart plugin version 171.4694.29

[✓] Connected devices
    • SAMSUNG SM N920A • 1115fbe3a1b01304 • android-arm • Android 5.1.1 (API 22)

This could also be #10481

@liyuqian is this closeable?

@cbracken I hope so but I can't find the note 5 in our device library for the confirmation. I'm asking to see where I can find it... In case we can't find it, shall we just close it as it's no longer reproducible?

I'll just go ahead and close this since there has been no update after the comments above. Feel free to reopen if it is still relevant.


After a panic() the terminal input does not show

Describe the bug
After a program panic, the terminal input does not work properly.
Setup
Please complete the following information along with version numbers, if applicable.

Ubuntu 22.04.4 (WSL2)
BASH 5.1.16(1)-release
Microsoft Terminal 1.20.11781.0
tmux 3.2a
bubbletea v1.1.0

To Reproduce
Steps to reproduce the behavior:

Execute the provided sample program.
Attempt to write to terminal after the panic, it will not display anything.

Source Code
package main

import (
	"fmt"
	"os"
	"time"

	tea "github.com/charmbracelet/bubbletea"
)

type model struct{}

func (m model) Init() tea.Cmd {
	return nil
}

func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
	return m, nil
}

func (m model) View() string {
	return "I will panic"
}

func main() {

	go func() {
		time.Sleep(2 * time.Second)
		panic("")
	}()

	m := model{}

	if _, err := tea.NewProgram(m).Run(); err != nil {
		fmt.Println("Uh oh, we encountered an error:", err)
		os.Exit(1)
	}
}

Expected behavior
I expected control to be given back to the terminal emulator and for it to be working normally.
Screenshots

After a panic the terminal behavior is broken, running reset fixes it.
Additional context
Add any other context about the problem here.

I'm also running into this. #846 may fix the issue, but it's been open for a while with no activity.

Thanks for the report! So I can reproduce this, however it's somewhat of an interesting case because the panic is occurring independent of Bubble Tea.
#846 was designed to solve a separate issue, but it may be what we want here. We'll look into this.

Ah, you're right, this panic isn't coming from inside a tea.Cmd. Sorry for confusing the situation!


Cannot get plugin to work

When user tries to pay, result is only error.
Wordpress 5.5.2
WooCommerce 3.7.0
Plugin 1.1.3
Server logs show following error.
AH01071: Got error 'PHP message: PHP Warning: Division by zero in /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-content/plugins/woocommerce-checkout-finland-gateway/src/Gateway.php on line 1133\nPHP message: PHP Fatal error: Uncaught Error: Call to a member function getName() on null in /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-content/plugins/woocommerce-checkout-finland-gateway/src/Gateway.php:712\nStack trace:\n#0 /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-content/plugins/woocommerce/includes/class-wc-checkout.php(924): CheckoutFinland\WooCommercePaymentGateway\Gateway->process_payment(2691)\n#1 /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-content/plugins/woocommerce/includes/class-wc-checkout.php(1124): WC_Checkout->process_order_payment(2691, 'checkout_finlan...')\n#2 /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-content/plugins/woocommerce/includes/class-wc-ajax.php(462): WC_Checkout->process_checkout()\n#3 /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-includes/class-wp-hook.php(286): WC_AJAX::checkout('')\n#4 /var/www/vhosts/mavacom.fi/cross4wd.mavacom.fi/wp-includes/class-wp-hook.php(310): WP_Hook->apply_filters('', Array)\n#5 /var/www/vhosts/mavacom.fi/cross4wd.mava...\n', referer: https://cross4wd.mavacom.fi/kassa/

@maavalko, there was an issue in tax calculation for non-taxable order items. This is fixed in 1.1.6.


Fix "verbose and redundant code constructs" findings

PR Description
Ran the "verbose and redundant code constructs" inspection in IDEA and it found these things. I think these are all simple, straightforward changes that help readability a little.
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

No worries, I can revert those changes. I wasn't a huge fan of them either.


Error MSB3030: All of these samples are broken and don't work with the new version of Visual Studio 2017

This is not well kept and is probably better not released than to release broken samples.
How do I fix this and what can be done
Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB3030	Could not copy the file "C:\repos\Windows-universal-samples\Samples\WebView\js\Debug\StreamUriResolverCPP\SDK.WebViewSampleCpp.winmd" because it was not found.	WebView	C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets	4222

Each sample was verified to work when it was published. I don't have the time to re-test all 500+ samples whenever a new version of Visual Studio comes out.
The issue you found appears to be a missing dependency. Rebuild again and it should work.

Okay, somehow they got unchecked from the "Any CPU" configuration. Switch to some other configuration (x64, x86, or ARM). "Any CPU" isn't really meaningful any more.

@oldnewthing thank you.  I would love to help but am not to familiar with all of the inner workins just yet.

@oldnewthing Just to advise.... Issues aren't usually closed until the update has been officially deployed.  Would you mind reopening the issue?

It's already in staging.


iron-form and POST method problem

Hello, i use Polymer starter kit 1.0.2 and i'm trying to use iron-form based on (little) documentation i found.
My method form is "post" and contain only one input.
My form "action" is a PHP script (add.php) showing content of $_GET and $_POST:
echo '
';
print_r($_POST);
print_r($_GET);
echo '
';

My form component (form_eclp.html) is:
<dom-module id="my-form">
    <template>

                    <form is="iron-form" id="formGet" method="post" action="add.php">
                        <paper-input name="name" label="Name" required></paper-input>

                        <paper-button raised onclick="clickHandler(event)">Submit</paper-button>
                    </form>

    </template>

</dom-module>

I call if from:
<link rel="import" href="form_eclp.html">
<my-form></my-form>

When i click the submit button after entering the text 'test' in name input, i can see in the network tab of the browser developper tools that it's a POST request, ok, but the url is add.php?name=test, and in the response tab i have:

Array
(
)
Array
(
    [name] => test
)

According to my form action (add.php script), first array is for $_POST and the second $_GET.
I can see, despite form method="post", it's a "get" request because only $_GET is populated, there is nothing in $_POST.
Have i missed something ?... i don't understand.


[Lib] Add sock_getpeeraddr method

Add sock_getpeeraddr method for TcpStream. In the meantime, some
attributes like Clone, Copy and Debug are added to structs. Also passed
clippy check.
Signed-off-by: KernelErr<EMAIL_ADDRESS>

Since there is a new API, could you please bump the library version?

Since there is a new API, could you please bump the library version?

Sure, should it be 0.2.1?

Do we change any existed API? If so, 0.3.0 to present the breaking changes. If not, 0.2.1 is fine.


Scaling to zero not working in KEDA 1.4.0

Scaling to and from zero is not working with Prometheus scaler after KEDA upgrade from 1.3.0 to 1.4.0.
Expected Behavior
KEDA should scale up deployment from zero when Prometheus metric increases from zero and scale down to zero when the metric decreases to zero.
Actual Behavior
KEDA scales deployment from zero to one replica even when Prometheus metric is zero and never scales it down to zero.
Steps to Reproduce the Problem

Deploy KEDA in GKE cluster using KEDA Helm chart in version 1.4.0.
Deploy any deployment and corresponding ScaledObject with minReplicaCount=0 and use Prometheus scaler with query that returns constant zero.
Watch that KEDA never scales the deployment down to zero.
Upgrade KEDA using the same Helm chart with KEDA downgraded to 1.3.0 using values:

image:
  keda: docker.io/kedacore/keda:1.3.0
  metricsAdapter: docker.io/kedacore/keda-metrics-adapter:1.3.0

and watch that KEDA scales the deployment to zero and again up from zero when the metric becomes positive as expected.
Specifications

KEDA Version: 1.4.0
Platform & Version: linux/amd64
Kubernetes Version: 1.15 (1.15.9-gke.24)
Scaler(s): Prometheus

Could you please rerun 1.4.0 with debug log leve on KEDA operator? https://github.com/kedacore/keda#setting-log-levels (or set it in the chart).

And paste here your ScaledObject please.

apiVersion: keda.k8s.io/v1alpha1
kind: ScaledObject
metadata:
  name: prometheus-scaling
  labels:
    deploymentName: worker
spec:
  scaleTargetRef:
    deploymentName: worker
  pollingInterval: 30
  cooldownPeriod:  60
  minReplicaCount: 0
  maxReplicaCount: 2
  triggers:
  - type: prometheus
    metadata:
      serverAddress: http://prometheus-server.monitoring.svc.cluster.local:9090
      metricName: queue_length
      threshold: '120'
      query: avg_over_time(worker_queue_length[5m])

I don't see anything suspicious in keda-logs.txt, except the last line
I0423 05:18:21.328041       1 wrap.go:47] GET /openapi/v2: (3.086429ms) 404 [ <IP_ADDRESS>:33094]
KEDA scales replica to 1 from 0 even though the Prometheus metric is zero:
{"level":"info","ts":1587618899.730634,"logger":"scalehandler","msg":"Successfully updated deployment","ScaledObject.Namespace":"staging","ScaledObject.Name":"prometheus-scaling","ScaledObject.ScaleType":"deployment","Deployment.Namespace":"staging","Deployment.Name":"worker","Original Replicas Count":0,"New Replicas Count":1}

It seems like this regression was brought by this change: https://github.com/kedacore/keda/pull/695/files#diff-a63ae5a2f6036b9f3bc750d5fe46437cR105
@droessmj what did you do this particular change? ie. Set scaler to active even for value 0?

@zroubalik  That commit updated a zero result to be non-error inducing. Based on the behavior described above I'm assuming since it's now not throwing errors, the scaler ensures 1 replica is up regardless. The "fix" I introduced just returns zero as a valid metric. If we need a special case where zero is non-error inducing but not a valid metric that can be introduced, but as-is the referenced commit just allows this code to now run for zero results:
	metric := external_metrics.ExternalMetricValue{
		MetricName: metricName,
		Value:      *resource.NewQuantity(int64(val), resource.DecimalSI),
		Timestamp:  metav1.Now(),
	}

@droessmj I see, but the scaler marks itself as Active even when there is zero result. So my only concern is the change in isActive() function: return val > -1, nil. This should be on the first look reverted back to return val > 0, nil. WDYT?

I believe you're correct. Reverting the isActive check while retaining the other part should resolve.

@hmoravec are you able to retest the change please if I send you link to dev image later today?

Would be great if anybody with Prometheus instance could check that the fix helped, just replace the images for KEDA Operator and KEDA Metrics Server. Thanks!
docker.io/zroubalik/keda:promFix
docker.io/zroubalik/keda-metrics-adapter:promFix

@zroubalik Sure, I'll test it. Btw automatic tests are planned? :)

We are always open for PRs ;)
But you are right, this shouldn't have slipped through. Sorry about this.

@zroubalik Working, thanks! It scaled down to zero when metrics became zero and scaled up when metrics became positive.


Show advice messages to help to fix lint error with error message

I have discussed in https://github.com/biomejs/biome/discussions/1390
react-hooks/exhaustive-deps shows advice message including concrete missing variable names with error message such as React Hook useEffect has a missing dependency: 'missingVariableName'. Either include it or remove the dependency array., but biome VSCode extension shows only error message such as This hook specifies more dependencies than necessary. when we use useExhaustiveDependencies rule.
With concrete missing variable names, we can quickly fix lint error. So I want you to consider about it.

@mehm8128 I moved it here because the diagnostics is something that part of the core repository, not the VSCode extension

Sorry, Thank you🙏

esLint have a quick fix for it, that either adds or removes the necessary dependencies.
Would it be possible to add that functionality?

esLint have a quick fix for it, that either adds or removes the necessary dependencies.
Would it be possible to add that functionality?

I think it's best to create a new issue for that. Creating a code action isn't as simple as changing a phrase in the diagnostic.

For this issue, is it ok that we only show missing or excessive variable names with error message such as This hook specifies more dependencies than necessary: 'excessiveVariable'?


[Keyboard, Web] Map from "Esc" to the Escape key

This PR fixes https://github.com/flutter/flutter/issues/106062 and adds the following entries to the Web key mapping:

Code "Esc" to physical key esc.
Key "Esc" to logical key esc.

The change to PhysicalKeyEntry is a bit tricky, since it only allows one chromeName. I didn't want to change chromeName to a list of string and cause hundreds of lines of changes in physical_key_data.json, so I added a new field otherWebNames.
The test is co-authored with @egramond from https://github.com/flutter/flutter/commit/58fa5754dc31dab90f0068615654a409f9be8fe2#diff-ac9c378fae8af8f4ffb68fad0b56e14f9f15ab797a0e7271a3690317a4df9c59.
All mapping changes will also be applied to the engine in a separate PR.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Looks great. Thank you!


feat: custom next-auth api endpoint path

support custom next-auth path (/api/tina/auth/*) to support consolidated api without rewrite

@logan-anderson allows the next-auth client to use the /api/tina/auth/* endpoint


Threading is not working ????

This simple code is not working:
from threading import Timer

def wakeUpCall():
    node.warn('time to wake up')

t = Timer(5.0, wakeUpCall)
t.start()
node.warn('waiting for timer to fire')

I can confirm this works with at least with my stepper motor task. 2 simple threads were spawned, did some gpio/driver type things and joined and completed fine.


Installation instructions are missing a wheel?

QwikContext
The Installation instructions seem to be missing a wheel that I have.
Details
After sh build.sh 1.2 345, there are two wheels created:
omdrivers-1.2.345_-py2.py3-none-any.whl   omsdk-1.2.345_-py2.py3-none-any.whl
Repro Steps

Grab the Zip file from the GitHub homepage
Unzip it
cd omsdk-master
sh build.sh 1.2 345
cd dist
You'll see 2 files there.
I performed:  pip install --user omdrivers-1.2.345_-py2.py3-none-any.whl omsdk-1.2.345_-py2.py3-none-any.whl

Impact
No problem for me.
NeedBy
Anytime. Is it even a problem? But then, if not, the additional file is spurious and confusing.
Severity
Low.

@GreyGnome, you are right in that there are two wheel packages created when you build omsdk. By default, pip install omsdk-<version>-py2.py3.none-any.whl will also install the omdriver wheel. So there is no need to install omdriver wheel package separately.


Rating not predicted today

Is it for only me or everyone?

for everyone

The worker was down. Its up now and should predict the rating for future contests. Closing the issue.


[Server] On delete author not cascading to quotes

When I delete an Author it's not cascading to the quotes that are related to this author. Right now the fix was place a try catch when trying to get the quote and not being able to find the Author of that quote.

The way that's being solved right now it's deleting the quote if a situation like that happens. This still need to be fixed to happen in database level.


[QUESTION] Token undefined?

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
When I try to run read_user_me on "/me" endpoint, an undefined (empty?) Token enters the function get_current_user (I saw it in debuger 'token='undefined') and after this I get "Authorization: Bearer undefined"
I definitely can’t understand if this is a bug, or an error in my code. Maybe I should use cookies...
python 3.8
last versions of libs
....
oauth2_scheme = OAuth2PasswordBearer(
    tokenUrl="/token",
    scopes={"me": "Read information about the current user."},
)

async def get_current_user(
    security_scopes: SecurityScopes,
    token: str = Depends(oauth2_scheme)
):
    pb_key = usecases.users.GetPublicKey().execute()
    if security_scopes.scopes:
        authenticate_value = f'Bearer scope="{security_scopes.scope_str}"'
    else:
        authenticate_value = f"Bearer"
    credentials_exception = HTTPException(
        status_code=status.HTTP_401_UNAUTHORIZED,
        detail="Doesn't work :'(",
        headers={"WWW-Authenticate": authenticate_value},
    )
    try:
        payload = jwt.decode(token, pb_key, algorithms=['RS256'])
        username: str = payload.get("sub")
        if username is None:
            raise credentials_exception
        token_scopes = payload.get("scopes", [])
        token_data = entities.TokenData(scopes=token_scopes, username=username)
    except (PyJWTError, ValidationError):
        raise credentials_exception
    user = usecases.users.ReadByUsername().execute(user_name=username)
    for scope in security_scopes.scopes:
        if scope not in token_data.scopes:
            raise HTTPException(
                status_code=status.HTTP_401_UNAUTHORIZED,
                detail="Not enough permissions",
                headers={"WWW-Authenticate": authenticate_value},
            )
    return user

app = FastAPI(default_response_class=JSONAPIResponse)

@app.post(
    "/token",
    response_model=entities.Token,
    tags=["auth"]
)
async def login_for_access_token(
    from_data: OAuth2PasswordRequestForm = Depends()
):
    token = users_usecases.AuthUser().execute(
        user_name=from_data.username,
        password=from_data.password
    )
    return token.value.dict()

@app.get(
    "/me",
    response_model=users_presenter.UserJSONObject
)
async def read_user_me(
    current_user: users_presenter.UserJSONObject = Security(
        get_current_user,
        scopes=["me"]
    )
):
    return JSONAPIConverter().serialize(current_user)
....

Additional context
I have JSON-API entities and classes of user cases for manipulating data from the database

This undefined does not look like Python generated. Did you test this with a browser (since you did not mention how you test this)?

this

I test this with a browser, of course. I changed my code and now I have:
async def login_for_access_token(
    from_data: OAuth2PasswordRequestForm = Depends()
):
    token = users_usecases.AuthUser().execute(
        user_name=from_data.username,
        password=from_data.password
    )
    return {"access_token": token.value, "token_type": "bearer"}

This code looks like the code from the offical tutorial. Perhaps the problem is parsing entities. At the output, I got JSON: API entity, and now just what was in the tutorial and it worked.

I think I should create new bug issue. I took the code from the official tutorial and replaced the output values for the endpoint:
@app.post("/token")
async def login_for_access_token(form_data: OAuth2PasswordRequestForm = Depends()):
    user = authenticate_user(fake_users_db, form_data.username, form_data.password)
    if not user:
        raise HTTPException(status_code=400, detail="Incorrect username or password")
    access_token_expires = timedelta(minutes=ACCESS_TOKEN_EXPIRE_MINUTES)
    access_token = create_access_token(
        data={"sub": user.username, "scopes": form_data.scopes},
        expires_delta=access_token_expires,
    )
    return {
        "data": {
            "id": "1",
            "types": "tokens",
            "attributes": {
                "value": access_token
            }
        }
    }

Perhaps there is no mistake and I just do not fully understand how the dependencies work.
I call @tiangolo to help me understand this strange code behavior.

I am having this same exact problem. The token endpoint isn't being called despite me having current_user: u.User = Depends(u.get_current_active_user) in my function kwargs.
On the docs page I can log in successfully using johndoe and secret, but then any other call to the API does not authenticate the logged in user as the token is undefined.

Bug raised at #1418

Thanks for the help here @phy25 ! :bow:
@idmitryz please provide a simple, self-contained example that shows your error, otherwise, it's very difficult to guess what might be wrong.

Actually, I'm noticing the same issue. Can anybody post how they solved it?

If it helps, I just ran into this and it was because the Swagger-UI was not adding the token to the Authorization header. It was doing this because I had the Token() model from the example in the docs but I was also using an alias generator that was camelCasing the response.
class Token(BaseModel):
    access_token: str
    token_type: str

    class Config:
        alias_generator = camel.case

https://stackoverflow.com/questions/59808854/swagger-authorization-bearer-not-send#:~:text=In the cURL request you,header%2C it cannot be found.

Hi @coryvirok , did you find a way to handle the token generation with this camel case alias, or did you have to return to original form
access_token: str  token_type: str

{
    "access_token": "token",
    "token_type": "bearer"
}

above works well.
the following results in  : -H 'Authorization: Bearer undefined'
{
    "errcode": 10000,
    "errmsg": "",
    "data": {
        "access_token": "token",
        "token_type": "bearer"
    }
}


Fix Zoom Level Limit issue

Make the properties not disappear as you zoom out. https://share.cleanshot.com/ymktG1jW
A user should be able to zoom out and see the status of properties across all of Philadelphia.

there are too many features in the dataset to render tiles at lower zooms.
in time I can explore other hosting solutions besides mapbox, but not something we can get done before the deadling
some ideas:

add a popup saying to zoom in to see properties
cluster into points at lower zooms
don't allow zooming out past where data is visible

I added the popup back and will close this ticket. we can re-open if we want to and have the bandwidth to explore further


Failed to execute 'btoa' on 'Window': The string to be encoded contains characters outside of the Latin1 range.

If edit a record within array what has some unicodes then occured exception with error page.
It's not occured neither below 4.0.0 and demo page.
Probably it's version 4's problem.
details are below.
Message: Failed to execute 'btoa' on 'Window': The string to be encoded contains characters outside of the Latin1 range.
Screenshots:

Closing because the fix is merged


Quit game from menu instead of transitioning to main menu.

When quit_requested is emitted from the QuitMenu, the UI closes the game. The player_died method now calls reset, which transitions back to the MainMenu same as before.
This fixes issue #29.
Please check if the PR fulfills these requirements:

[x] The commit message follows our guidelines.
For bug fixes and features:

[x] You tested the changes.
[x] You updated the docs or changelog.

Related issue (if applicable): #29

What kind of change does this PR introduce?
Does this PR introduce a breaking change?
New feature or change
What is the current behavior?
What is the new behavior?
Other information

Thanks much!


:warning: always use up-to-date context

We were using stale context. I observed it when looking at tracer output, some spans were being recorded in the scope of main context, instead of their provider specific context.
:warning: changes provider api

Could there be a time when the engine is run multiple times? I think that is the use case?

@shawn-hurley I don't think I completely understand, as long as we pass the same context to engine and all providers here https://github.com/konveyor/analyzer-lsp/blob/6ad9f5e29e94ab2c8080c690cf56c5610bb7de08/cmd/analyzer/main.go#L164 and here https://github.com/konveyor/analyzer-lsp/blob/6ad9f5e29e94ab2c8080c690cf56c5610bb7de08/cmd/analyzer/main.go#L136. shouldn't that be fine?

Yeah, but could there be a time when the engine is running on its own, and rules come in, and the providers are instantiated when the rules come in (if not set)? Stoping one call to then engine then is different than stopping the full engine.
You are not wrong that right now, this will work exactly as you expect, I think keeping them separate prevents some hard debugging in future IMO
If we don't think that this is ever a concern that it is not a huge deal.

makes sense, then I am good with this change

@pranavgaikwad can you rebase?

@shawn-hurley rebased.

Sorry need another rebase, I am sorry I tried to reduce this


Implement asynchronous cleanup of unreferenced blob data

A process should lock and sweep registry data to garbage collect unreferenced. https://github.com/docker/distribution/issues/422#issuecomment-97165192 covers many of the issues will implementing such a solution. During this time, the registry may be in down time or read only mode.
#461 would be a prerequisite of this implementation.

cc @ncdc

This would be really useful for us at Shopify. We had a hacked-together GC script for our V1 registry but haven't bothered to write anything for V2 yet. Our growth is a little unsustainable though and we're definitely going to need the ability to prune old blobs sooner or later. (cc @Sirupsen)

In the meantime, a bash script that can delete a v2 registry image, metadata and blobs, if you are storing data on the local filesystem (not s3).

@burnettk That looks great. I am surprised to haven't seen more implementations like that. We're working on a read only mode to make this style of GC easier to implement, which should work nicely with that script. It would be great to get updates on your experience with the script so we can inform the GC implementation.

@stevvooe cool. the script deleted 30GB of the 100GB of data we were using in one hour in its inaugural run. i had it delete the 400 oldest of our 1200 images (we don't really tag, but just create new images/repos with each build). no sign of data corruption yet. :)

@burnettk Great to hear!

we don't really tag, but just create new images/repos with each build

I really don't recommend creating new image names for every build. The registry is designed more to handle a high cardinality of tags, over a high cardinality of repositories. You'll also pay an extra price in bandwidth to push the layers to a new repository each time.

You'll also pay an extra price in bandwidth to push the layers to a new repository each time.

Just to be sure I understand how the content-indexed storage works: if one were to push an indentical stack of layers to a new repository in the same registry, would there be any transfer required? Would it be different if it were a new tag in the same repository?

@jonathanperret This is more about the security model than content-addressed storage. For a layer to be available in a given repository, the client must prove it has the data before it is accessible through a given repository. This requires a transfer of the data into the repository, which is verified. A new tag in the same repository does not require this transfer.
This is required to prevent layer ids from becoming a "secret", as is the case in v1 of the protocol. Otherwise, simply knowing a given layer id would give any client access.
An optimization, #634, is in the works to avoid this case in the future.

+1 for this. I've got few remarks:

Wouldn't it be more client-friendly to do the pruning on a blob immediately (or with some delay) when it gets unreferenced? It would require to keep a graph of objects in memory though. Each guarded with atomic reference counter to avoid global locking. Are there any drawbacks apart from code's complexity?
Could there be an alternative in form of some pruning API?

e.g. POST */vacuum/pruneBlobs - on demand lock world and remove unreferenced blobs
e.g. POST */vacuum/recursiveRemove/<repo>/<name>?tag=*

2nd option favors scenarios where a system of record lies outside of registry.

@miminar The main issue with such an approach is that it requires consensus. I'd like to avoid any extra APIs as we can likely do without. Please see the road map for details on deletes.

+1

@MohsenBS Please avoid extraneous commentary. We understand this is a needed feature and it is in the works. If you would like to get updates, you can subscribe to notifications on this issue.

@stevvooe
I've been looking over the issues and constraints for deletion as I have experienced some storage problems and housekeeping came up as a concern.
Would it be feasible to consider some model of kv (file) store where the key could be the blob-id and the value a list of the manifests/images that reference this blob?
In such scenario the item would be created/updated(append) when a new image that references the blob is pushed and updated when an image that references the blob is deleted (remove the image from key's value).
GC could look for empty keys and one by one lock them (mechanism TBD), remove the blobs and remove the keys.
Thank you for your time!

@realreadmedotcom GC could look for empty keys and one by one lock them (mechanism TBD), remove the blobs and remove the keys.

The issue is keeping the set of "empty" keys consistent with the set of items. Without transactions, what we thought was an "empty" key may become referenced again in the meantime, invalidating the ability to delete. The problem is that the GC may encounter a new blob before it gets added to the reference list. The GC would mark it for deletion.
The other issue is the failure of dependent writes. For example, what if one writes the blob but the update to the reference list is not? What if two writes are made to the same bucket, resulting in a data race where one list addition is not added? In both cases, the GC will delete data that was intended to be kept. Even higher-level, where do you hold the locks?
Both of the above issues can be solved by transactions. Let's take adding a new blob reference:
tx := begin()
tx.add(blob)
references := tx.add(manifest)
increment(references)
commit(tx) // must apply all changes or none

Note that we are using reference counting in this example, since content addressable data is always a DAG, but I hope the concept is clear. Deleting a manifest:
tx := begin()
references := tx.delete(manifest)
decrement(references)
commit(tx)

Now, we have a set of references, with counts. To collect blobs that are no longer referenced, we do the following:
tx := begin()
deletable := find(blob joined with references where count=0)
tx.delete(deletable)
commit(tx)

With such an approach, we could have fairly safe online GC. The requirements are that transactions are completely serializable and that we have the ability to calculate references from a provided document (ie manifest).
I hope this clarifies the problem, in general.
If you're curious about the concept of GC, Go's blog has a very gentle introduction to the concept. They couple concurrent, tri-color mark and sweep with mutation capture while a GC is ongoing. While not super easy to implement, understanding the solution lends itself to understanding the main problem. It is very different from the example above (it handles cycles and other conditions), but it would be equivalent to keeping track of every remove and delete while sweeping all manifests.
BTW, we've merged an implementation of the mark and sweep collector. It does require the registry backend to be locked in some way, but it is ready for use.

closed by #1386

@RichardScothern I am not sure if #1386 covers the "lock" portion of this issue. We may want to keep another issue open so that important detail isn't lost.

Hi,
I may understand this issue all wrong, but I thought scenarios like this simple example should be covered by this one. (Running this example in boot2docker)
1/ pull two different images:
docker pull distribution/registry:master docker pull ubuntu
2/ Run registry:
docker volume create --name registry docker run -d -p 5000:5000 -p 5001:5001 \ -v /root/registry-config.yml:/etc/docker/registry/config.yml \ -v registry:/var/lib/registry \ --name registry \ distribution/registry:master
3/ Tag those two images as "test:lates" and push them:
docker tag distribution/registry:master localhost:5000/test docker push localhost:5000/test docker tag ubuntu localhost:5000/test docker push localhost:5000/test
4/ Now there should be unreferenced blobs in my registry right? Stop registry. Run garbage collection:
docker stop registry  docker run \ -v /root/registry-config.yml:/etc/docker/registry/config.yml \ -v registry:/var/lib/registry \ --rm distribution/registry:master garbage-collect /etc/docker/registry/config.yml
5/ the blobs were not deleted
Am I missing something here?
Thanks!

@stevvooe : #1386 does not enforce locking  (though the documentation in that PR does mandate it).  Enforcing locking/read-only mode across a registry cluster would be a sizable undertaking and I think the move toward live GC is a more worthwhile goal.

@vehovmar Please open another issue.

@stevvooe why another, shouldn't this one be re-opened? Or am I misunderstanding this issue and the use case I posted should not be covered by this one?

@vehovmar This issue is a feature request and you are looking for support of the feature. What you describe may be a bug or it could be completely unrelated.
This is our policy and I appreciate you for following it.

All right @stevvooe here it is #1600. Thanks!