Add Monero Address

Apologies if this is in the wrong repository. I will repost if directed.
I would like to see support for a Monero address in the Keybase client. Monero is a private, fungible cryptocurrency that has been around since 2014.
According to this blog post, the following is what Keybase is looking for in a cryptocurrency:

Zcash fixes the privacy issues, while retaining the usability we want: just post an address on your profile once, and that's it. Let the money flow in.

Monero has the same functionality. One can post any Monero address, and it is private by default. If you would also like to allow visibility (in relation to "tip jars or donations"), you can also ask users for an optional view key that will show the incoming transactions.
Monero has a substantially larger market capitalization than Zcash, and it has a larger community. I believe if Keybase found enough interest for Zcash, there is also enough interest for Monero.
Learn more at getmonero.org. If you are having difficulty adding Monero, please visit the Reddit page or #monero-dev on IRC.

Other cryptocurrency issues: #5788 (reusable payment codes), #7042 (altcoins), #7560 and #4266 (Ethereum).

Zcash is already added as an altcoin, so I see no reason other coins should not be supported.

I second this request. Is there something blocking this issue, something we might help with?

Bump - There is still interest in this request.

Agreed, whats the status on this? Lots of people really want this feature.

If you want to be extremely innovative, you can allow only Monero subaddresses to be posted :)

Adding my voice to the chorus.  Please add Monero support!

keybase have added ZCash and Grin before adding Monero (even adding Grin while the protocol is going through inevitable growing pains).
There must be some reason why Monero is not added, and i would hazard a guess the reason is not entirely technical.

Would be nice to get even a response then, if there's some underlying issue blocking this.

Yes, please add Monero support!  Or at least let us know what the blocking issues are.  Thanks!

Please add Monero!

I use keybase almost daily and would really appreciate monero support. Thank you.

XMR FTW!

I think the problem with adding Monero support is that they have a sort of aggressive and fanatical community that oftentimes goes out of their way to attack other coins. It may be created some "bad blood" between other coins to add Monero. ZCash is relatively benign in that regard. At least imo.

@duckrocks let me provide an alternative take on this.
The Monero community is very passionate about privacy, and we all acknowledge that real privacy is incredibly hard to get right. Those of us in the Monero community even realize that Monero has several weaknesses in regards to our own privacy.
What is very frustrating is when coins with gigantic claims to fame about their "impenetrable privacy" pop up out of the wood work all the time. Projects like Verge, TokenPay, and more. These projects usually offer incredibly weak privacy at best, and their "marketing claims" (that are just trying to make them a quick buck) are very dangerous to anyone who believes them.
The Monero community tends to refute these claims, swiftly and without apology, and in so doing, Monero has gotten a (hilarious) reputation amongst other coins as being "trolls" and "party poopers", just because Monero has the audacity to rain on the parades of the moon kids by telling them that their privacy isn't up to snuff.
It'd be similar to someone rolling their own crypto (noticeably and verifiably badly) and putting it in a chat app and claiming that you'll be "safe and encrypted forevermore" and something like Signal comes in and says "no, that's not how encryption works" at which point all the people working on the theoretical app (or otherwise invested) say that Signal goes out of their way to be aggressive and fanatical to other encryption apps because it wants to stay on top.
There has indeed been some bad blood at times between Monero and Zcash, although many in the community acknowledge our goals are more aligned than many might think, it's just our implementations and ideals are drastically different. This was clearly evidenced by the cross-country privacy panel that occurred when MoneroKon and Zcon1 were taking place at the same time on different parts of the world: https://www.youtube.com/watch?v=DWPELgmS2uA
In summary, while Monero can often times be passionate about their approach to correcting bad information, I think its erring into hyperbole to say the majority of the community is aggressive and fanatical.

Ah ok, I understand. I do not mean to insult the entire Monero community, obviously you do good work. What I was merely getting at was there have been more than one occasion where I or a colleague have done research on various cryptocurrency related topics, and when coming in the privacy sphere seeing some, I don't know how else to put it, "bad behavior".
And this was usually from certain, select members of the Monero community, and yes it was usually in relation to the privacy of other coins. I think its fair to put it down to zeal or passion or whatever, but all I meant was that I can see how it could possibly make it politically more difficult to add XMR on a platform because of this. Some may see it and be turned off from the coin even though it was merely the behavior of a few loose cannons and zealous guys.
This is all I say, and it is just my limited opinion. I do not mean to come across as attacking you or your coin.

Would be nice, especially if it actually supported a wallet, not just address.  How much cash does it take? I got about tree fiddy.

If you add Monero, it would be very cool!

Add XMR! ;)

Monero is definitely important to add.

I agree, Monero (XMR) should be added to Keybase ASAP, as it is the closest coin to Satoshi's true vision of 1-CPU, 1-vote due to its ASIC resistance, and it is also uniquely fungible due to its enforced privacy by default, unlike Bitcoin and the vast majority of altcoins.

Please add Monero support!

Best fit considering the nature and solid development of both projects.
Love to see this happen.

upvote

+1 👍👍


Returning score from text search

Is it possible to return the full text score from a Query?
This is what my raw query looks like:
db.disease.find( { $text: { $search: "brain" } },
                          { score: { $meta: "textScore" } } )
          .sort( { score: { $meta: "textScore" } } )

This works correctly and includes score in the results as well as sorting the results by score.
This is the same query using the MongoDB Java driver directly.
// search with the Java driver
BasicDBObject textSearch = new BasicDBObject("$search", "brain");
BasicDBObject search = new BasicDBObject("$text", textSearch);

BasicDBObject meta = new BasicDBObject("$meta", "textScore");
BasicDBObject score = new BasicDBObject("score", meta);

List<DBObject> diseases = collection.find(search, score).sort(score).toArray();
Assert.assertEquals(2, diseases.size());
Assert.assertEquals("brain", diseases.get(0).get("name"));
Assert.assertEquals("benign-brain", diseases.get(1).get("name"));

I've been reading through the documentation and I can't figure out how to accomplish the same thing in Morphia. From the search example in the Morphia documentation (http://mongodb.github.io/morphia/1.0/guides/querying/#text-searching), this is how my query looks:
List<DiseaseBean> diseases = datastore.createQuery(DiseaseBean.class)
                                      .search("brain")
                                      .order("_id")
                                      .asList();
Assert.assertEquals(2, diseases.size());

This returns the correct documents, however does not include the score and is ordering by "_id". I don't see any way to handle the $meta operator in Morphia to get at the "textScore". Is this possible?

I see this was also posted to stackoverflow.  I'll answer there since the issue tracker isn't the best forum for this kind of question.
Answered:  https://stackoverflow.com/questions/30709505/include-score-in-morphia-full-text-search/30738588#30738588


[SMALLFIX] Fix Table of Contents for Project Docs

There was a comparator in the if statement that was using !== instead of != so undefined and null were not resulting in equal values, causing the scrollspy plugin to fail.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4389/Failed Tests: 1org.tachyonproject:tachyon-integration-tests: 1tachyon.master.MasterFaultToleranceIntegrationTest.faultTest
Test FAILed.

+1 LGTM


ionice not available on Solaris

Hi,,
This line breaks gitlab_git on solaris. ionice is not available on Solaris and should not be used the same way it's not used on "darwin" kernels.
https://github.com/gitlabhq/gitlab_git/blob/8192622ac919fb0c58eb4bc652d8785252a596f2/lib/gitlab_git/repository.rb#L932
>> Gem::Platform.local
=> #<Gem::Platform:0x30a78c @cpu="x86_64", @os="solaris2.11", @version=nil>
>> Gem::Platform.local.os
=> "solaris2.11"

[root@gitlab /home/git]# uname -a
SunOS gitlab 5.11 joyent_20131009T194502Z i86pc i386 i86pc Solaris

/cc @randx

@krzaczek can you propose a fix?

@randx  I'm not a rubby guy so I won't send You a PR ... but it looks like You need to modify line https://github.com/gitlabhq/gitlab_git/blob/8192622ac919fb0c58eb4bc652d8785252a596f2/lib/gitlab_git/repository.rb#L931 and exclude solaris the same way You exclude darwin kernels

@maxlazio can you look into it?

Yes, MR is here: https://gitlab.com/gitlab-org/gitlab_git/merge_requests/25

MR has been merged in 4ce7663c00670a7c3f7e5e6ce8d1f7541f51881b.


Add support for blink it and swiggy instamrt.

I have just started using the plugin and it's very cool.
Blink it also has a web ui. Adding support to blink should be straight forward. I am not sure how we can add support for swiggy instamart.
If you don't have the bandwidth, I can raise a PR for supporting blinkit. LMK.

Hi @anmol5varma,
I personally don't use Blinkit. Please feel free to make a PR to add it.
Also, thanks a lot for your interest!


How to expand specific node

I can enumerate through all my nodes in the tree this way:
var nodes = angular.element('#tree-root-' + website.id + " [ui-tree-node]");
for(var i = 0; i < nodes.length; i++) {
    var node = nodes[i];
}

Now I want to get the scope of the node and maybe expand it. How is that possible? node.expand() is undefined, node.scope() too.

Do you want to iterate all your node and expand ? If not, you can use this,
$scope.collapseAll = function() {
  $scope.$broadcast('collapseAll');
};

$scope.expandAll = function() {
  $scope.$broadcast('expandAll');
};

Hi Nobal, thank you for your answer.
I know these broadcast events, but as you already note, they expand or collapse the whole tree. I only want to expand/collapse specific single nodes.
I need this to implement a search function. When searching with a string, I collapse the whole tree and then expand only the matching nodes.

Got it. I think, you can follow this approach, its a nasty hack

Call a custom expand function for each node. ng-click="CustomExpand(this)"
CustomExpand will be triggered only on true of some variable, say "SearchStringFound". ng-click="SearchStringFound || CustomExpand(this)"
Iterate the tree and if matching search node found. Set the SearchStringFound to true
Set $scope.SearchStringFound = false onload

In controller,
$scope  CustomExpand = function(scope) {
scope.expand();
//if not $nodeScope.expand();
}

I had this problem today as well, this was my approach:
I basically iterate the whole tree, using the childNodes() function until I find the requested node, collecting the parent scopes along the way.
I then expand all the ancestors and the requested node (it's no use expanding a node if its parent is collapsed).
I have a plunker here: http://plnkr.co/edit/5EQu0mtpaOQOuJtLeH0t?p=preview
Don't know if there is an an easier/better way though.

@mschuler @sebver I think for any given node, the combination of depth, index is unique (                 scope.$nodeScope.depth(), scope.$nodeScope.index() ). So, you can use them for condition checking in view and trigger a click event to expand !

Thank you @sebver, your plunker helped me to solve my problem!

Why might the following line of code return an empty set when the tree is visible in the UI?
return angular.element(document.getElementById("tree-root")).scope().$nodesScope.childNodes()[0]
My problem is published here: https://stackoverflow.com/questions/47933196/how-to-expand-a-default-angular-ui-tree-node

I got @sebver's version working with a little tweaking.
In getScopePathIter I had to change
var childNodes = scope.childNodes();
to
var childNodes = scope.$element[0].children[1].children;
I did that because the original was giving an array where all the elements were the same.  This made it give me an array of html elements.
Then I changed the for loop from
for (var i = 0; foundScopesPath === null && i < childNodes.length; i++) {
           foundScopesPath = getScopePathIter(nodeId, childNodes[i], newParentScopeList);
       }

to
for (var i = 0; foundScopesPath === null && i < childNodes.length; i++) {
                var childNode = angular.element(childNodes[i]).scope();
                foundScopesPath = getScopePathIter(nodeId, childNode, newParentScopeList);
            }

so it would take the html elements and get their corresponding scopes.
Hope that's helpful.

HI, matt,
i have problem suppose i have a page which user can select and then save so
after save page will reload, in case its not working properly if am going to hard refresh then work

This issue would be trivial if there was a getNodeScopeById API.


chore: update api

chore: update api
chore: release

#1  👈
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


[📑 Docs]: Improve 'streetlights' tutorial

What Dev Docs changes are you proposing?
Proposed change
The current 'streetlights tutorial needs improvement; the content outline must be restructured to match the new tutorial outline template we'll be using for our docs moving forward.
The below section provides the new sample outline template to use for all of our tutorials, and I've added some basic building blocks in this example to help make it relatable...

NEW agnostic tutorials outline  (sample template below)
Introduction
In this AsyncAPI and Apache Kafka tutorial, we're going to learn about...
Background context
Apache Kafka is an open‑source distributed event-streaming platform. Similar to AsyncAPI, it uses an EDA architecture for...
Installation guide
Before you proceed to the next stage, you'll need to download a few things:

Install Node.js (v15 or newer).
Install Git on your Operating System (OS):

Install Git on MacOs
Install Git on Windows
Install Git on Linux

Tutorial step 1
In this step, we will...
Tutorial step 2
In this step, we will...
Tutorial step 3
In this step, we will...
Summary:
In this tutorial, we built a...
Next steps:
Now that you've completed this tutorial, why don't you check out how to do ABCs with our ABCs tutorial? You may also want to check out the XYZ tutorial to learn how to XYZ.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@starlightknown While you await more responses to your User Survey to identify new tutorials to add, here is your first TW gsod issue. ⭐
Notice how @Annysah will be working on a parallel assignment, except she's doing this work for the 'Getting Started' tutorial. Feel free to learn from each other!

awesome, I'll start working on this. thank you


After https is configured, the website cannot enter the page on the mobile phone. Just keep refreshing

Describe the bug
After https is configured, the website cannot enter the page on the mobile phone. Just keep refreshing
配置了 https后网站在手机端无法进入页面。就只是不停的刷新
Reproduction
minimal reproduction : https://github.com/rashagu/httpsTest
Configure vite.config.js
import { defineConfig } from 'vite'
import vue from '@vitejs/plugin-vue'

// https://vitejs.dev/config/
export default defineConfig({
  server:{
    https: true,
  },
  plugins: [vue()]
})

System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (4) x64 Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz
    Memory: 6.61 GB / 15.91 GB
  Binaries:
    Node: 14.17.4 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.11 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.14 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.73)
    Internet Explorer: 11.0.19041.906

Used Package Manager
yarn
Logs
no  error log

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

When running the command yarn dev are you passing in the host option? Ex: yarn dev --host <IP_ADDRESS>
Also on your mobile device are specifying https://<your device's network address>:3000 when typing in the address?
I was able to recreate the issue only when I didn't specify the https:// in the network address. Both for Chrome and Firefox on a Pixel 5. Then the same for iOS (though, not the latest version or recently patches iPhone 8 Plus,)
Commands I entered in the Node.js command prompt after a new install of Nodejs 14.17.4 on Windows 10.
yarn create vite httpsTestProj
cd httpsTestProj
yarn
yarn dev --host <IP_ADDRESS>

System Info:
  System:
    OS: Windows 10 10.0.19043
    CPU: (20) x64 Intel(R) Core(TM) i9-10900KF CPU @ 3.70GHz
    Memory: 23.87 GB / 31.92 GB
  Binaries:
    Node: 14.17.4 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.11 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.14 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.78)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    @vitejs/plugin-vue: ^1.3.0 => 1.4.0
    vite: ^2.4.4 => 2.5.0

Possible duplicate of #2337

Indeed looks like a duplicate of #2337


Can we get the acces of the official documentation of the C2pa

@sagarmahto905 The specifications for C2PA are fully public at https://c2pa.org/specifications/specifications/2.0/specs/C2PA_Specification.html.   What other material would be useful to you?


Provide a uniform api between Simple, KeyValue and Chain stores

Provide a uniform api between Simple, KeyValue and Chain stores for public and private methods.
This is because a couple of gems I use out there need to call some of the internal methods of the simple store. When you try to use other gems configured with a Chain backend call those calls fail.
Basically I added the initialized?, translations and init_translations to the KeyValue and Chain stores.
Also added the functionality to be able to read back the translations from a key value store back to a hash as if it where read from the YAML files.
The gems that i know that would benefit from this would be i18n-js and babilu

Any further updates on this? Just bumped into the issue with i18n-js; I'm probably going to work around it for now but this would be useful in the future.

@kidpollo Yes I like this idea too. Could you please update your branch against the latest master? Will look at releasing this in 0.9.0.

Woah 5 years later still relevant?

@kidpollo Do you think it would be relevant? Is it still useful to provide a uniform API?

@radar updated

Still seems useful if you want to make another attempt at getting it cleaned up and merged.

Updated again

Thank you for keeping this continuously up to date. I have a few comments for you, and then I think this is good to merge :)

Done @radar

Thank you!
I will merge this on Monday when I am back at work.


Merge master branch to 3.x dev branch

What is the purpose of the change
Sync changes between these two branches.
Brief changelog
merge and resolve conflicts
Verifying this change
No commits lost.
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3718 into 3.x-dev will decrease coverage by 0.21%.
The diff coverage is 78.6%.

@@              Coverage Diff              @@
##             3.x-dev    #3718      +/-   ##
=============================================
- Coverage      63.78%   63.57%   -0.22%
+ Complexity        75       71       -4
=============================================
  Files            656      703      +47
  Lines          28454    31001    +2547
  Branches        4812     5022     +210
=============================================
+ Hits           18150    19708    +1558
- Misses          8015     9024    +1009
+ Partials        2289     2269      -20

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/dubbo/config/MetadataReportConfig.java
92.3% <ø> (+51.28%)
0 <0> (ø)
:arrow_down:

...er/configurator/override/OverrideConfigurator.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...nnotation/AbstractAnnotationConfigBeanBuilder.java
93.33% <ø> (ø)
0 <0> (ø)
:arrow_down:

...notation/ReferenceAnnotationBeanPostProcessor.java
94.36% <ø> (+17.98%)
0 <0> (ø)
:arrow_down:

...ava/org/apache/dubbo/rpc/cluster/Configurator.java
92.3% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../annotation/DubboConfigConfigurationRegistrar.java
100% <ø> (ø)
0 <0> (?)

.../org/apache/dubbo/config/spring/ReferenceBean.java
18.62% <ø> (-16.62%)
0 <0> (ø)

...hema/CompatibleAnnotationBeanDefinitionParser.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...g/context/properties/DefaultDubboConfigBinder.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...beans/factory/annotation/ReferenceBeanBuilder.java
70.58% <ø> (-16.6%)
0 <0> (ø)

... and 353 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02cca09...e6f473b. Read the comment docs.


Fixed height of mat-form-field even without mat-label or mat-hint

Bug, feature request, or proposal:
Fixed height of mat-form-field even without mat-label or mat-hint
What is the expected behavior?
Without any mat-label or mat-hint defined inside mat-form-field with appearance standard, fill or outline,
mat-form-field's height should be reduced and content should be centered.
What is the current behavior?
Height of mat-form-field is not changed. Label is just hidden at the reserved place. Same thing for mat-hint.

What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://stackblitz.com/edit/angular-urvaf8-4b9lc1
What is the use-case or motivation for changing an existing behavior?
New fill appearance is used by Google on new product like Google Agenda. No floating label, and height of component is reduced.

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All
Is there anything else we should know?
Thanks a lot. Hope this report will help.

The issue with doing this is that if you placed a form field with a hint and one without next to each other, they wouldn't align anymore.

Would it be too much if we had some more attributes like nolabel nohint? I kind of understand @tfalvo's fustration (I haven't have a situation like his, but this flexibility would be welcome).

Thank you for your reply!
I understand completely the problem.
However, as a developer, it's our responsibility to ensure the coherence of design, and so to be sure to respect same height/same appearance for each form field of the same group/page.
What do you think about it?
But I know also that material design spec doesn't mention this "compact" appearance. But Google doesn't respect itself the spec, and use a lot of this new appearance, in addition to this, with a leading icon at the left (and not inside the box)... I like this new layout... what a pity this is not officially mentioned in spec.

I concur with this, especially because the non-compact version does not fit nicely into a mat-toolbar.
In my case I am trying to use:
<mat-form-field appearance="outline">

If anyone wants something like an input that is smaller (but not perfect), then see my code below. Sorry don't have time to make a demo.
<mat-toolbar [color]="themeService.toolBarColor" class="mat-elevation-z2">
  <mat-form-field appearance="outline" fxFlex>

    <button mat-button fxFlex>
      <input matInput placeholder="P" [matAutocomplete]="autop" [(value)]="selectedP.name"
[formControl]="pControl"
        matTooltip="Search available P">
      <mat-icon [matMenuTriggerFor]="pMenu">arrow_drop_down</mat-icon>
    </button>
    <mat-menu #pMenu="matMenu" xPosition="after">
      <ng-template matMenuContent>
        <ng-container *ngFor="let outer of nestedArray">
          <ng-container *ngFor="let inner of outer.p">
            <button mat-menu-item (focus)="true" (click)="onClick();">
              {{inner.name}}
            </button>
          </ng-container>
        </ng-container>
      </ng-template>
    </mat-menu>

    <mat-autocomplete #autop="matAutocomplete" [displayWith]="displayFn">
      <mat-option *ngFor="let item of filteredOptionsP | async" [value]="item" matTooltip="{{item}}"
        (onSelectionChange)="doSomething()">
        {{item}}
      </mat-option>
    </mat-autocomplete>

  </mat-form-field>
</mat-toolbar>

In the css file:
mat-form-field { /* Changes the input styling to just about nothing so that button styling can take over */
  font-size: 1px;
}

/* ::ng-deep .mat-form-field-appearance-outline .mat-form-field-outline {
  color: transparent !important; //remove border if needed
} */

.mat-button {
  padding: 0 30px; /* Changes position of dropdown icons */
}

With some tweaking, this allows me to still follow the Material Design Spec (https://material.io/design/navigation/search.html#usage) for a component that does not exist in the available angular material library.

I also support this idea. We also have separate labels (not in the input element), and in most places, no hint or error text. Therefore it takes way too much space, and it seems to be difficult or feels like a hack to overcome that.
Additionally, it might be a nice idea if the space is only taken in error-case (configuration).

According to https://material.io/components/text-fields/#outlined-text-field, the height stays the same even when there's no label. This part could be improved by adding a "dense" attribute, but shrinking should not be the default behavior.
On the other hand, the vertical alignment is clearly an issue and should be fixed.

Here is a quick fix applied to the global CSS:
.mat-form-field:not(.mat-form-field-has-label) .mat-form-field-infix {
  border-top-width: 0;
}

It might not cover all the use cases, especially with prefix/suffix icons, but I hope this helps.

I think the current padding (which is 1em) in the mat-form-field-infix class does not follow the spec : https://material.io/components/text-fields/#outlined-text-field.
Did anyone noticed it ?

@adrogon fix + wrapper setting below made the input box "normal" for me...
.mat-form-field:not(.mat-form-field-has-label) .mat-form-field-infix {
  border-top-width: 0;
}

.mat-form-field-wrapper {
  padding-bottom: 0px;
  margin-bottom: 0px;
  margin-top: 0px;
}

Thanks!

Related to #5227

Using @adrogon solution, I adjusted the top of the infix slightly and have nice alignment with matPrefix and matSuffix:
.mat-form-field:not(.mat-form-field-has-label) .mat-form-field-infix {
   top: -0.2em;
}

according to material spec https://material.io/components/text-fields both filled and outlined text field can have dense spacing, when this will be supported in angular material?

Here is a quick fix applied to the global CSS:
.mat-form-field:not(.mat-form-field-has-label) .mat-form-field-infix {
  border-top-width: 0;
}

It might not cover all the use cases, especially with prefix/suffix icons, but I hope this helps.

I worked for me but adding the prefix ::ng-deep

Based on the comments in this thread and my own testing, I think it's important we get an update for this. It doesn't seem like it's possible to implement even a basic search field without a label and achieve symmetric vertical spacing without custom css. That seems like a major flaw in this implementation of the material design spec.

Any update on this? I think dense text fields can be very interesting

Please add dense. It will be very helpful

The availability of a dense setting would save so much time and headaches.

Hello, is there any update regardig this issue?

Another viable use-case would be to use mat-form-field for styling an input but using the input as a text filter for a list/table.

any updates on this?

This is possible with the new MDC-based form-field. There are a few things you need to do:

Apply the density setting for the size form-field you want, e.g. @include mat.form-field-density(-5);
Add a placeholder to your <input>
Don't put a mat-label on your form-field, instead specify the label via aria-label on the <input>. This ensures that the placeholder will be shown

Example: https://stackblitz.com/edit/angular-u9e74t-m8y9m8?file=src%2Fapp%2Finput-form-example.html,src%2Ftheme.scss


add "plugin" system to voyager

first design will be implemented via an optimization pass, hooking into the existing framework for that. this is an incredibly powerful (and complex) tool for relayer operators, allowing for arbitrary inspection and modification of the queue.

closed by #2661


⚠ Turnos 24x7 has degraded performance

In 326ecca, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 6166 ms

Resolved: Turnos 24x7 performance has improved in c04271c.


changes how post it notes are added, now u can add post it notes by right clicking the background.

I like this, works as expected.


HttpClient interception

Hey guys,
first of all, thank you for the awesome work you do with Blazor.
What is or what would be the recommended way to reconfigure the HttpClient in the DI? Are Delegating Handlers supported at all?
I need support for the following use cases:

Send CSRF token with each POST/PUT/… request like Angular does. (or are there any other way to achieve protection any other way?)
Intercept all responses and redirect the currently logged-in user to login, when session expires and the backend API returns 401.
Retry for particular error codes.

Thank you.

Based on BrowserServiceProvider.cs:
serviceCollection.AddSingleton(new HttpClient(new BrowserHttpMessageHandler())
{
    BaseAddress = new Uri(BrowserUriHelper.Instance.GetBaseUri())
});

I have found a workaround, so I can reregister HttpClient with all the handlers added:
public void ConfigureServices(IServiceCollection services)
{
    services.AddSingleton(_ => new HttpClient(
        new AuthenticationDelegatingHandler(
        new CsrfDelegatingHandler(
            new BrowserHttpMessageHandler()
        ))
    )
    {
        BaseAddress = new Uri(BrowserUriHelper.Instance.GetBaseUri()),
    });
}

Obviously for GA that would be great if we could have the same simplicity like ASP.NET Core's new HTTP extensions has, so that you only have to add `DelegatingHandler˙s to the DI.

I think your existing solution with registering an HttpClient or a Func<HttpClient> in DI is a good one. We'll consider adding further parity with ASP.NET Core server-side APIs in the future.

Based on BrowserServiceProvider.cs:
serviceCollection.AddSingleton(new HttpClient(new BrowserHttpMessageHandler())
{
    BaseAddress = new Uri(BrowserUriHelper.Instance.GetBaseUri())
});

I have found a workaround, so I can reregister HttpClient with all the handlers added:
public void ConfigureServices(IServiceCollection services)
{
    services.AddSingleton(_ => new HttpClient(
        new AuthenticationDelegatingHandler(
        new CsrfDelegatingHandler(
            new BrowserHttpMessageHandler()
        ))
    )
    {
        BaseAddress = new Uri(BrowserUriHelper.Instance.GetBaseUri()),
    });
}

Obviously for GA that would be great if we could have the same simplicity like ASP.NET Core's new HTTP extensions has, so that you only have to add DelegatingHandler's to the DI.

This is no longer relevant. Is there any way of using DelegatingHandler or MessageProcessingHandler with the current HttpClient/Http pipeline?

M

You can use WebAssemblyHttpMessageHandler instead of BrowserHttpMessageHandler

WebAssemblyHttpMessageHandler works as expected! Thank you! Is there documentation anywhere for this?

I didn't find any documentation, and I met another problem, I would like to inject my "CustomService" into "CustomDelegatingHandler" to make functional intercepting, but cant find nice solution

This is about as clean as I could get it: https://gist.github.com/dylanberry/4a24df4fb369770e43b8a3e34997f854
Hope that's helpful.


[JOSS Review] Scenario comparison example

The docstring of the scenarios argument in Experiment does not give enough information, I have to go to the code to understand how it works. Additionally, I would highly recommend including the "scenario comparison" functionality in one of the example notebooks.

Hi @martibosch, thank you for pointing this out.
Indeed, the scenarios argument has not made it into any of the demonstration models
as it has proven more straight-forward to represent scenarios through different parameter combinations.
It will be depreciated in the upcoming release of v0.1.0.


Small XML library enhancements

Noticed that XML library can be enhanced a little while demonstrating it to one team. These are all somewhat minor enhancement and decided to submit just a single issue about them:

[x] Save XML should log where the XML was written.
[x] Element To String should accept optional encoding to ease passing results forward as bytes.
[ ] Should document and test that possible doctype is preserved only when using lxml.
[x] Should document that comments are always stripped.
[x] Avoid FutureWarning when using lxml.

While working with this, I noticed that processing instructions totally fail parsing when using lxml (#2146). That will be fixed by stripping PIs similarly as we already strip comments (#1748). Once this issue is fixed we also document that comments and PIs are stripped.


Add article number to complex source

fixes https://github.com/antleaf/nims-mdr-development/issues/613

@anusharanganathan Thank you!


Bump to version 0.13.0

r? @philipc

I'll publish on crates.io and make a tag once this is rubber stamped and merged, assuming there aren't reasons we shouldn't make a release right now that I'm not thinking of.

Coverage remained the same at 89.054% when pulling 49e6eccd98291b8050c3ae9bab8a65d4969b0249 on fitzgen:bump-to-0.13.0 into e2271620b63319a7513c738281764fd869980be8 on gimli-rs:master.

New version published on crates.io! Thanks @philipc :)


Adding skonsole workflow to generate PR descriptions.

Motivation and Context

This adds a PR comment-triggered workflow that uses the action mkarle/skonsole-generate-pr-description to run skonsole on the diff of the PR.  That action uses mkarle/pr-update to update the PR with a title, a summary, and a list of changes.  It can update as a prefix, suffix, or entirely replace what's there.
Description

To use, create a comment on a PR with:
/sk generate-pr-description <suffix(default) | prefix | replace>
Since it's triggered from a pull request comment, it only runs if it's in the default branch of the repo. See this PR for an example of it working.
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

@microsoft/octo-semantickernel-pr-apps FYI

This is really cool! It would be nice to see the prefix and/or suffix options in action as well.

This is really cool! It would be nice to see the prefix and/or suffix options in action as well.

In the PR I linked in the description I used replace, and it defaults to suffix. There's an example of prefix in this one


SSH Root Password

I was able to flash the image and get an SSH connection, but when I try to log in as root, it asks for a password. I've tried 'onhub' and several others as well as no password. No joy.

Default password is "changeme". It's mentioned on flashing instruction page if you read it carefully.

 I figured it out earlier today. I was able to get in change the password, Busybox and I'm going to try and install openVPN. Did you end up installing anything? Thanks for the reply!

I am currently struggling with galeforce being overridden by Google WiFi firmware update. It's says in the description that it should "survive" firmware update but apparently it does not. Something has changed since it was published 2 years ago,


couchdb_os_daemons_tests fails assertEqual

Expected Behavior
couchdb_os_daemons_tests:187: should_die_quickly should pass...
Current Behavior
    couchdb_os_daemons_tests:187: should_die_quickly...*failed*
in function couchdb_os_daemons_tests:'-check_dead/2-fun-3-'/2 (test/couchdb_os_daemons_tests.erl, line 231)
in call from couchdb_os_daemons_tests:check_dead/2 (test/couchdb_os_daemons_tests.erl, line 231)
in call from couchdb_os_daemons_tests:should_halts/2 (test/couchdb_os_daemons_tests.erl, line 210)
**error:{assertEqual_failed,[{module,couchdb_os_daemons_tests},
                     {line,231},
                     {expression,"D # daemon . status"},
                     {expected,halted},
                     {value,running}]}

Jenkins run:

https://builds.apache.org/blue/organizations/jenkins/CouchDB/detail/master/87/pipeline/49

Again @ https://builds.apache.org/blue/organizations/jenkins/CouchDB/detail/master/92/pipeline/49

Fixed by #679. Will reopen if we see this recur.


Allow users to provide the kubernetes provider they would be running Uffizzi on

Tell us about your request
Make it possible for users to mention which kubernetes provider are they installing Uffizzi on.
uffizzi install --provider gke
uffizzi install --provider vanilla

The above should default the default UffizziClusters to have the provider passed down to the following UffizziCluster spec.
kind: UffizziCluster
spec:
  provider: vanilla

If the provider flag is not given, the CLI should install with the "vanilla" provider. So that users can easily install Uffizzi on minikube. All clusters provided will then be vanilla k3s by default. If users provider gke then all UffizziClusters created should have the following default spec.
kind: UffizziCluster
spec:
  provider: gke

The above spec will be picked up by the Uffizzi Cluster Operator and the correct nodeselctor and tolerations spec will be given to the Workloads.
Which service(s) is this request for?
cli, app, controller
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Installing and creating clusters on minikube/k3d
Describe the solution you'd like
Mentioned in the request
related to #503

This is fine, just change provider to node-selector and change gke to gvisor. Don't bother with vanilla, just leave null as the default.

updating this issue with new changes about nodeselector and tolerations


Allow dps values for tuya powerstrips

Hello!
Is it possible to implement the dps option for tuya powerstrips like they were available in https://github.com/codetheweb/homebridge-tuya?
Thank you!
Carsten

Hi,
I just pushed "CustomMultiOutletAccessory" that lets you do this. This is how you use it:
{
  "name": "Custom Outlet",
  "type": "CustomMultiOutlet",
  "manufacturer": "GeekBee",
  "model": "Smart WiFi Power Strip",
  "id": "xxxxxxx",
  "key": "xxxxxxxx",
  "outlets": [
    {
      "name": "Heat Gun",
      "dp": 1
    },
    {
      "name": "Solder Iron",
      "dp": 3
    }
  ]
}

Until I make a release, you can update the plugin with npm i -g AMoo-Miki/homebridge-tuya-lan and play with it. Lemme know how it works or if you have any questions.

Hello!
I tried this:
            {
                "name": "Wohnzimmerdoppeldose",
                "type": "CustomMultiOutlet",
                "manufacturer": "Cotify",
                "model": "Smart Wifi",
                "id": "xxxxx",
                "key": "yyyyy",
                "outlets": [
                    {
                        "name": "Weihnachtsstern",
                        "dp": 1
                    },
                    {
                        "name": "Repeater unten",
                        "dp": 2
                    }
                ]
            },

and what happened is this:
——
Mar 14 16:27:27 HomePi homebridge[17049]: [3/14/2019, 4:27:27 PM] [TuyaLan] Connected to Wohnzimmerdoppeldose
Mar 14 16:27:27 HomePi homebridge[17049]: [3/14/2019, 4:27:27 PM] [TuyaLan] Connected to Kamera
Mar 14 16:27:27 HomePi homebridge[17049]: [3/14/2019, 4:27:27 PM] [TuyaLan] Ready to handle Wohnzimmerdoppeldose with signature {"1":false,"2":true,"9":0,"10":0}
Mar 14 16:27:27 HomePi homebridge[17049]: /usr/local/lib/node_modules/homebridge-tuya-lan/lib/BaseAccessory.js:34
Mar 14 16:27:27 HomePi homebridge[17049]:             const nameCharacteristic = service.getCharacteristic(Characteristic.Name) || service.addCharacteristic(Characteristic.Name);
Mar 14 16:27:27 HomePi homebridge[17049]:                                                                  ^
Mar 14 16:27:27 HomePi homebridge[17049]: ReferenceError: Characteristic is not defined
Mar 14 16:27:27 HomePi homebridge[17049]:     at CustomMultiOutletAccessory._checkServiceName (/usr/local/lib/node_modules/homebridge-tuya-lan/lib/BaseAccessory.js:34:66)
Mar 14 16:27:27 HomePi homebridge[17049]:     at device.context.outlets.forEach (/usr/local/lib/node_modules/homebridge-tuya-lan/lib/CustomMultiOutletAccessory.js:35:31)
Mar 14 16:27:27 HomePi homebridge[17049]:     at Array.forEach ()
Mar 14 16:27:27 HomePi homebridge[17049]:     at CustomMultiOutletAccessory._verifyCachedPlatformAccessory (/usr/local/lib/node_modules/homebridge-tuya-lan/lib/CustomMultiOutletAccessory.js:29:37)
Mar 14 16:27:27 HomePi homebridge[17049]:     at CustomMultiOutletAccessory._registerCharacteristics (/usr/local/lib/node_modules/homebridge-tuya-lan/lib/CustomMultiOutletAccessory.js:49:14)
Mar 14 16:27:27 HomePi homebridge[17049]:     at TuyaAccessory.BaseAccessory.device.once (/usr/local/lib/node_modules/homebridge-tuya-lan/lib/BaseAccessory.js:22:18)
Mar 14 16:27:27 HomePi homebridge[17049]:     at Object.onceWrapper (events.js:285:13)
Mar 14 16:27:27 HomePi systemd[1]: homebridge.service: Main process exited, code=exited, status=1/FAILURE
Mar 14 16:27:27 HomePi systemd[1]: homebridge.service: Unit entered failed state.
Mar 14 16:27:27 HomePi systemd[1]: homebridge.service: Failed with result 'exit-code'.
Mar 14 16:27:37 HomePi systemd[1]: homebridge.service: Service hold-off time over, scheduling restart.
Mar 14 16:27:37 HomePi systemd[1]: Stopped Node.js HomeKit Server.
Mar 14 16:27:37 HomePi systemd[1]: Started Node.js HomeKit Server.
——

Am 13.03.2019 um 09:30 schrieb Miki<EMAIL_ADDRESS>Hi,
I just pushed "CustomMultiOutletAccessory" that lets you do this. This is how you use it:
{
"name": "Custom Outlet",
"type": "CustomMultiOutlet",
"manufacturer": "GeekBee",
"model": "Smart WiFi Power Strip",
"id": "xxxxxxx",
"key": "xxxxxxxx",
"outlets": [
{
"name": "Heat Gun",
"dp": 1
},
{
"name": "Solder Iron",
"dp": 3
}
]
}
Until I make a release, you can update the plugin with npm i -g AMoo-Miki/homebridge-tuya-lan and play with it. Lemme know how it works or if you have any questions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/AMoo-Miki/homebridge-tuya-lan/issues/14#issuecomment-472326449, or mute the thread https://github.com/notifications/unsubscribe-auth/AqTkfPn-wk9FFZEXG_XCj3CBR0xzxJGoks5vWLcwgaJpZM4bZbg6.

I accidentally deleted a vital line after testing and before pushing it. I have fixed it now. I was way too sleepy when I wrapped this up. Let's hope that was the only thing I messed up.

Yes, now it works like a charm! Thank you very much for this!

Am 14.03.2019 um 23:23 schrieb Miki<EMAIL_ADDRESS>I accidentally deleted a vital line after testing and before pushing it. I have fixed it now. I was way too sleepy when I wrapped this up. Let's hope that was the only thing I messed up.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/AMoo-Miki/homebridge-tuya-lan/issues/14#issuecomment-473088746, or mute the thread https://github.com/notifications/unsubscribe-auth/AqTkfNRPqTlKrdMl7tyGYChyzCrmQyA0ks5vWsvhgaJpZM4bZbg6.

Awesome. Thanks for confirming.


[MODFISTO-298] Partial rollover

Purpose
MODFISTO-298 - Unable to access adjustment ledger
Approach

New test of a rollover with no order type selected for encumbrances
New reusable features
Fixed a permission issue

On snapshot


"model 'gpt-4o' not found, try pulling it first"

I got this error.
openai.NotFoundError: Error code: 404 - {'error': {'message': "model 'gpt-4o' not found, try pulling it first", 'type': 'api_error', 'param': None, 'code': None}}
I am using the example code - what am I doing wrong?
https://docs.gptr.dev/docs/gpt-researcher/pip-package
(Also i used os.environ["OPENAI_API_KEY"] to set my openai key)

This is strange. Did you try exporting your key like so: export OPENAI_API_KEY=... ?


Create windows arm64 release archive

Windows arm64 artifacts are already being produced and we need to start creating a win arm64 bundle. The following are the required steps:

[ ] https://github.com/flutter/flutter/issues/143230
[ ] Update prepare_package script create the windows arm64 archive and add it to the metadata file.
[ ] Create a package_builder build to test the configurations running on a windows arm64 machine.
[ ] Update website to include the new archive

There are no blocker for main/master branch but there are two blockers for release candidate branches:

Dart to start producing win arm64 artifacts for branches different than main.
Availability of win arm64 machines in dart-internal.

@CaseyHillers can you please help triage and reassign?

@itsjustkevin FYI
@yusuf-goog can you please help us identify the requirements to add win arm64 machines to dart-internal
@sortie can you please help us identify if there are any blockers on dart to produce the sdk for non main/master branches?

@godofredoc Our dart-sdk-win-arm64-stable builder has been lived since December, the full set of channels are available for win arm64 :) It does do a cross build though rather than running on a win arm64 cloud instance. We don't have win arm64 cloud instances in dart-internal, but we do have linux arm64 cloud instances. We will need a disk image for windows arm64 to spawn these bots if you need them.

I looked at the new T2A arm machine types from GCP, and spoke to chromeops, they said that they do not have a win arm image either, and that they believe it would not be a simple/quick fix if they need to set one up.
As for them hosting some physical machines for us, I'm following up on that.
Though it seems after speaking to @athomas, builds are cross compiled right now, so we may not need these hosts?

If it can help, at Linaro, we ran windows-arm64 VM using kvm and wrote an install tutorial.

@rmacnak-google helped build the disk image and luci support for our linux-arm64 cloud instances. Maybe he could help with information beneficial to the windows arm64 cloud effort?


pac4j-saml: Typecast error in SAML2DefaultResponseValidator

There is a typecast error in SAML2DefaultResponseValidator.validateAssertionSignature().
Look at the following lines:
     final QName role = selfContext.getRole();
        if (signature != null) {
            final String entityId = peerContext.getEntityId();
            validateSignature(signature, entityId, engine);
        } else if (((SPSSODescriptor) role).getWantAssertionsSigned()
                && !peerContext.isAuthenticated()) {
            throw new SAMLException("Assertion or response must be signed");
        }

The cast of role to SPSSODescriptor is not valid. Role is of type QName and QName does not have any common part of hierarchy with SPSSODescriptor.
Once the execution gets to the else-if condition (if the signature is null), this code will crash. This stacktrace is from 1.7.1 but valid for 1.8.x too:
Caused by: java.lang.ClassCastException: javax.xml.namespace.QName cannot be cast to org.opensaml.saml.saml2.metadata.SPSSODescriptor
	at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateAssertionSignature(SAML2DefaultResponseValidator.java:659)
	at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateAssertion(SAML2DefaultResponseValidator.java:400)
	at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateSamlSSOResponse(SAML2DefaultResponseValidator.java:307)
	at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validate(SAML2DefaultResponseValidator.java:151)
	at org.pac4j.saml.sso.impl.SAML2WebSSOMessageReceiver.receiveMessage(SAML2WebSSOMessageReceiver.java:83)
	at org.pac4j.saml.sso.impl.SAML2WebSSOProfileHandler.receive(SAML2WebSSOProfileHandler.java:54)
	at org.pac4j.saml.client.SAML2Client.retrieveCredentials(SAML2Client.java:253)
	at org.pac4j.saml.client.SAML2Client.retrieveCredentials(SAML2Client.java:77)
	at org.pac4j.core.client.BaseClient.getCredentials(BaseClient.java:222)
	at org.jasig.cas.support.pac4j.web.flow.ClientAction.doExecute(ClientAction.java:196)
	at org.springframework.webflow.action.AbstractAction.execute(AbstractAction.java:188)
	at org.springframework.webflow.execution.ActionExecutor.execute(ActionExecutor.java:51)
	... 50 common frames omitted

I only have limited knowledge of OpenSaml but I think we should first find a RoleDescriptor for the QName and then call getWantAssertionsSigned() on it.

Hi,
Thanks for reporting this issue on the SAML support. Would you mind opening
a new Github issue for that?
I planned to release the version 1.8.0 this Thursday, I hope we'll be able
to fix that quickly.
Thanks.
Best regards,
Jérôme
2015-10-06 17:59 GMT+02:00 Jaroslav KAČER<EMAIL_ADDRESS>
There is a typecast error in
SAML2DefaultResponseValidator.validateAssertionSignature().
Look at the following lines:
 final QName role = selfContext.getRole();
    if (signature != null) {
        final String entityId = peerContext.getEntityId();
        validateSignature(signature, entityId, engine);
    } else if (((SPSSODescriptor) role).getWantAssertionsSigned()
            && !peerContext.isAuthenticated()) {
        throw new SAMLException("Assertion or response must be signed");
    }

The cast of role to SPSSODescriptor is not valid. Role is of type QName
and QName does not have any common part of hierarchy with SPSSODescriptor.
Once the execution gets to the else-if condition (if the signature is
null), this code will crash. This stacktrace is from 1.7.1 but valid for
1.8.x too:
Caused by: java.lang.ClassCastException: javax.xml.namespace.QName cannot be cast to org.opensaml.saml.saml2.metadata.SPSSODescriptor
at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateAssertionSignature(SAML2DefaultResponseValidator.java:659)
at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateAssertion(SAML2DefaultResponseValidator.java:400)
at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validateSamlSSOResponse(SAML2DefaultResponseValidator.java:307)
at org.pac4j.saml.sso.impl.SAML2DefaultResponseValidator.validate(SAML2DefaultResponseValidator.java:151)
at org.pac4j.saml.sso.impl.SAML2WebSSOMessageReceiver.receiveMessage(SAML2WebSSOMessageReceiver.java:83)
at org.pac4j.saml.sso.impl.SAML2WebSSOProfileHandler.receive(SAML2WebSSOProfileHandler.java:54)
at org.pac4j.saml.client.SAML2Client.retrieveCredentials(SAML2Client.java:253)
at org.pac4j.saml.client.SAML2Client.retrieveCredentials(SAML2Client.java:77)
at org.pac4j.core.client.BaseClient.getCredentials(BaseClient.java:222)
at org.jasig.cas.support.pac4j.web.flow.ClientAction.doExecute(ClientAction.java:196)
at org.springframework.webflow.action.AbstractAction.execute(AbstractAction.java:188)
at org.springframework.webflow.execution.ActionExecutor.execute(ActionExecutor.java:51)
... 50 common frames omitted
I only have limited knowledge of OpenSaml but I think we should first find
a RoleDescriptor for the QName and then call getWantAssertionsSigned() on
it.
—
Reply to this email directly or view it on GitHub
https://github.com/pac4j/pac4j/issues/306.

Hi Jerome, but this is a GitHub issue, isn't it? :-)

Sorry. Yes, it is ;-)

Working on this today. Stay tuned.

Hi, I have submitted a PR that should fix the NPE thrown from the else-if condition:
https://github.com/pac4j/pac4j/pull/316
It's not crashing anymore.
HOWEVER, I'm not completely sure the code in the else branch is actually useful. In my test case (CAS against Shibboleth, assertion signing switched off), the SP Descriptor read from the SAML2MessageContext is null. Does this make sense? I think the original purpose of this code was to read the SP Descriptor (non-null) and get the WantAssertionsSigned flag from it, then throw an exception if the requirement is not fulfilled.

Many thanks. Looping @mmoayyed
If Misagh is ok, I'll merge this today, retest tonight and release (yes !) tomorrow.

the SP Descriptor read from the SAML2MessageContext is null. Does this make sense?

No it doesn't. I am OK with the null check, but I don't think that ever should be null. Can you investigate why that context happens to be null? You should always have the SP Descriptor.

I will investigate later when I have more time...
Unless it's initialized by OpenSaml, I usually get lost there.

I think the PR generally is fine, but if you have a null context, you might have other issues with the integration. So definitely need to look into that.

@mmoayyed : can I merge and release? And investigation results on this point will be handled in version 1.8.x?

Yes, that's the plan. I think one additional change that would go into 1.8.x is to throw exceptions when that context is null because that should not really happen, but what we have now is perfectly safe for the release, so fire away :)

@jkacer : feel free to open a new issue for the "null context" issue and mark it for 1.8.1.
Many thanks again for your help.

Sure, I will when I find something relevant.
And you're welcome :-)


signature verification issues error code:110775b5d101fb36bc6c911beb774491d9ff06e2

See #564.

Fixed in torbrowser-launcher 0.3.4


Fix for cygwin setup on 64 bit.

Cygwin was having issues because it uses the win32 jni_md.h which was not compatible.  I bypassed this in the case were native jni.h was found with my own typedefs.  However, in the testing framework the native one was not found sending us to the fallback which of course did not take the same type of bypass.  Thus I have hardcoded the cygwin to always take the fallback removing the jlong incompatibility.

I think it is better to include jni.h only where it is really needed. Is there a good reason to do it in jpype.h? This is a very central spot.

It was hidden one layer lower in jp_utility.h.  Thus either way we were getting jni.h.  All I was doing was moving it one level higher exposing the fact that we pulled that in everywhere.  We do need the types declared in jni.h, but we don't need anything else except for two of the wrapper files.  I am not sure how we would get only the types.

I can certainly move it back to the original place, but I moved it up when I had to grep to figure out how anything was getting jlong.  Call it a style thing but I don't like really huge headers like that pulled in several layers deep.  It we are pulling in something big and nasty like everywhere which we did indirectly (jpype.h -> jp_utility.h -> jni.h), we should do it at the top level (jpype.h -> jni.h).

ah I see, jp_utility.h is also included in jpype.h

Okay I am going to have to mess with the appveyor pattern a bit.  For some reason we are not consistently getting JAVA_HOME set up thus we are often failing to build our test suite.  That is what triggered the jlong issue in the first place.  Fixing the jlong problem just pushed it farther down in the script where ant failed to build the required harness.
I need to add more checks into install.sh so that we fail much earlier if something vital is wrong or we will get lead on wild goose chases like this.

On 27.03.2018 23:07, Karl Nelson wrote:

Okay I am going to have to mess with the appveyor pattern a bit.  For some reason we are not consistently getting JAVA_HOME set up thus we are often failing to build our test suite.  That is what triggered the jlong issue in the first place.  Fixing the jlong problem just pushed it farther down in the script where ant failed to build the required harness.
I need to add more checks into install.sh so that we fail much earlier if something vital is wrong or we will get lead on wild goose chases like this.

Thanks for digging into this. So not having the right jni.h in place can
cause these issues?
Having more sanity checks more upstream sounds like a really good idea.

Yes, the base cause of the issue was that cinst put java into a different place and appveyor.yml had a hard coded path.  This caused the build process to try to build without a JAVA_HOME set which triggered a whole bunch of code paths that I don't evaluate locally.  Had it been clear what the original error was we would have fixed it in the test harness.
That said my original code was a hack that should not exist and I am sure others triggered the bug just like the test harness did.  So fixing it to use the same code path every time is likely the better solution.
I would like to try to work the following into the test pattern:

Don't do a separate appveyor for with and without numpy.  Instead it should build both and test them.  Appveyor jobs are really slow and the actually build and test is a small portion of the whole.
Test multiple versions of java in the same pass.  (same reason as previous)
Bomb proof the process so that it bails quickly if something can't proceed so it doesn't get stuck doing useless work.

Unfortunately, appveyor gets stuck so often that trying to debug it remotely is a real pain.  The appveyor job queue insists on fully testing even if a patch has been superseded.  It is going to take me a very long time before I could get those patterns fully tested.   I will likely have to push in a test patch with all tests disabled but one to speed up development.

This is why I asked @originell to give me Appveyor rights, which he did? in the meantime, so we can enable the rolling build option. This one cancels builds for superseded patches/commits.
I don't know about the implications of merging separated job configs into one build run. I mean the whole idea of having separated build boxes is to avoid messed up configurations. We should use a tool like tox or conda-build (which I'd prefer) to set up build matrices in one box. As this tool ensures (at least does a good job) we have the exact configuration we want.

I will defer to you on the testing matrix.  I believe that I can have more than one jpype module built in the same run so something simple like trying with one switch or another could be in the same build.  I have multiple builds (win-py2. win-py3, cygwin-py2, and cygwin-py3) in the same directory on my development machine.
I agree it is nicest if we don't have to manage any special switching in the build process but instead use a pure matrix.  However, if we really want to test (win,cygwin) x (x86, 64) x (2.7,3.5,3.6) x (Jdk7, Jdk8, Jdk9) x (w/o numpy, w/ numpy), it is just going to be too big of a matrix for appveyor.  It I force multiple tests together reducing this to (win,cygwin)x(x86,64)x(2.7,3.6) with evaluation of (Jdk7, Jdk8, Jdk9)x(w/o numpy, w/ numpy), then at least the matrix is reasonable.   It does mean calling ant once for each version of java, and then calling nose for each of those builds.
If you can switch the pattern to use conda-build as an example it would certainly help.  CI is not something I do much and tends to be more of a distraction for me.

Okay getting much closer.  We are still getting a random failure which is not reproducible in the appveyor environment that is preventing the cygwin from finding jvm.dll.   And we have successfully replicated the threading error on cygwin python2.  The obvious question being was python2 built with thread support.

@marscher you should've just gotten an email from appveyor, I hope :) added you with nearly all rights. Hope it helps. If not, I'll try to be super responsive today so we can move quick on this

I have now tried virtually every combination that I can think of.  I will have to try to find a way to replicate the problem.  The mystery to me is I know that we have had this working in the past.  Given that we never specified the correct 32bit jvm with the 32bit python, how could this have ever worked?

Okay if the 32 bit wasn't bad enough when I disabled that test the second test gave a false negative...
testPublicMethodPropertyOverlap (test.jpypetest.PropertiesTestCase) ... ok
Test multiple proxy calls with different threads at the same time. ... Fatal Python error: This thread state must be current when releasing
XXX lineno: 220, opcode: 0
XXX lineno: 220, opcode: 0
java.lang.RuntimeException: Python exception thrown: TypeError: attribute name must be string, not 'type'
	at jpype.JPypeInvocationHandler.hostInvoke(Native Method)
	at jpype.JPypeInvocationHandler.invoke(JPypeInvocationHandler.java:10)
	at com.sun.proxy.$Proxy1.testMethodObject(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
java.lang.RuntimeException: Python exception thrown: SystemError: unknown opcode
	at jpype.JPypeInvocationHandler.hostInvoke(Native Method)
	at jpype.JPypeInvocationHandler.invoke(JPypeInvocationHandler.java:10)
	at com.sun.proxy.$Proxy1.testMethodObject(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
java.lang.RuntimeException: Python exception thrown: SystemError: unknown opcode
	at jpype.JPypeInvocationHandler.hostInvoke(Native Method)
	at jpype.JPypeInvocationHandler.invoke(JPypeInvocationHandler.java:10)
	at com.sun.proxy.$Proxy1.testMethodObject(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
XXX lineno: 220, opcode: 0
XXX lineno: 220, opcode: 0
java.lang.RuntimeException: Python exception thrown: SystemError: unknown opcode
	at jpype.JPypeInvocationHandler.hostInvoke(Native Method)
	at jpype.JPypeInvocationHandler.invoke(JPypeInvocationHandler.java:10)
	at com.sun.proxy.$Proxy1.testMethodObject(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at jpype.proxy.ProxyExecutor$ProxyCaller.call(Unknown Source)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
[thread 1504 also had an error]
[thread 2432 also had an error]
[thread 1036 also had an error]
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x00000003fdae783a, pid=2560, tid=0x000000000000032c
#
# JRE version: Java(TM) SE Runtime Environment (8.0_162-b12) (build 1.8.0_162-b12)

Something is bothering me with the window anaconda 32 bit test.
Downloading JDK from http://download.oracle.com/otn-pub/java/jdk/8u162-b12/0da788060d494f5095bf8624735fa2f1/jdk-8u162-windows-x64.exe
Why is it using the 64 bit jvm?

@marscher I have disabled all of the failed tests so that we have a baseline configuration for the CI.  I placed an issue in the tracker to mark the defective platforms for now.  Assuming that the latest build passes we should squash the branch and merge it in.  We can then rebase some of those outstanding pulls to see what if anything is broken.  I am leaving the unsquashed here so you can browse the failures to see the failure modes.

Thanks for your effort here. Is Cygwin really such a common platform? I mean we have Anaconda for Windows, which provides tons of packages (not restricted to Python itself).
Did you disable Travis in this PR?

I use cygwin mostly as a development platform because it is separate from my production line which is running on anaconda.  The debugging tools are better for tracking down segfaults.  I think I have seen several other users post issues regarding the cygwin so it is certainly getting used though likely just as a poor mans unix.  That said if something is having problems on cygwin it is not the end of the world.  Those that happen on 64 bit/python 3.6 will get cleaned up as those are the paths I test, but if another contributor wants to keep the rest going I am fine with it.
I don't see anything in my changes that should have disabled Travis.  It was certainly not intended.

Ok. Dunno why travis is not being executed any more (eg to many builds, huh?). Maybe somebody is volunteering to setup cygwin. But it seems to be quite a hassle.


autocommit = off/on.

When I do autocommit on, will it retrieve the data from disk or from memory?
If I want to retrieve last 1000 values from memory and the rest from disk, should I commit once in 1000 new values ?
In general what is the memory consumption of the dict in RAM ?

Nothing is stored in memory, sqlite is a disk (file) based database.
The transaction is committed to disk with commit(), or after each operation if you have autocommit on. See official sqlite docs about the commit semantics (sqlitedict is just a wrapper for sqlite).

But what will happen if I add 10 new keys to the dict, with autocommit off,
Will those keys be in-memory?  accessing them is like accessing memory not disk ?

Sqlitedict will do whatever Sqlite itself does -- there's no additional transactional logic on top. See the link above, or ask at the Sqlite forum if in doubt.


Update for PureScript 0.15

This upgrades the code and dependencies for PureScript 0.15. Note: this shouldn't be merged until https://github.com/garyb/purescript-codec/pull/10 is merged and there is a new release.

Since 2303d37 built, I went ahead and updated the Bower file for the next codec release so that this can be merged without further changes after that merges.


Applied formatting to bokeh datetime hover tool labels

As requested in #1261 datetimes in the hover tool should be formatted correctly. This has annoyed me myself a fair bit. Currently the only approach to make this work is to send the formatted strings as a column in the ColumnDataSource as outlined here, in future we can hopefully use JS datetime formatting to support this. I do think this is an important feature (it's visible in a number of my demos like the StockExplorer) and should make it into 1.7.
See https://github.com/bokeh/bokeh/issues/1239 for status of a more convenient/efficient mechanism for this.

File an issue if you think there may be cleaner way to handle this in future. In the meantime I'll merge this PR given that the tests are passing.

File an issue if you think there may be cleaner way to handle this in future. In the meantime I'll merge this PR given that the tests are passing.

Not until bokeh do something about it.


⚠ AsianCinema has degraded performance

In 56985c7, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 1133 ms

Resolved: AsianCinema performance has improved in 126e2ed.


GDR2 files challenging download

Here is what I do to download the files from google drive. The id=... indicates the file identifier generated by Google Drive.
wget --no-check-certificate -r \
    "https://drive.google.com/uc?export=download&id=1fShusAHd3pjCXofixhcloHD-2XImz_zh"
     -O "gdr2.full.hdf5"

Note that Vaex does not raise any issue but returns a NoneType object when trying to open the file.

I downloaded the file directly from google drive and I everything worked fine. The file downloaded using wget is not a hdf5 file.

How do you retrieve the file from command line? That is my only option

I did not. I downloaded the file using Chrome. I searched the web to find how to do it using wget or curl but nothing worked out.

Seems that the following script worked.
# identifier on google drive for the 1.2T file.
ggID='1fShusAHd3pjCXofixhcloHD-2XImz_zh'
ggURL='https://drive.google.com/uc?export=download'
filename="$(curl -sc /tmp/gcokie "${ggURL}&id=${ggID}" | grep -o '="uc-name.*</span>' | sed 's/.*">//;s/<.a> .*//')"
getcode="$(awk '/_warning_/ {print $NF}' /tmp/gcokie)"
curl -Lb /tmp/gcokie "${ggURL}&confirm=${getcode}&id=${ggID}" -o "${filename}"

adapted from answer 41 in
https://stackoverflow.com/questions/25010369/wget-curl-large-file-from-google-drive?noredirect=1


Sonarqube issue cleanup

The Sonarqube report suggests a lot of issues that could be rectified for code quality and vulnerabilities. Here is the report:
Bugs found: https://jenkins.cs.ox.ac.uk/sonarqube/project/issues?id=mdm-ui&resolved=false&types=BUG
Security hotspots: https://jenkins.cs.ox.ac.uk/sonarqube/security_hotspots?id=mdm-ui
Clean up the following:
Security hotspots

[x] All security hotspots should be removed (currently only 3)

Critical bugs

[x] Remove all critical issues found. These relate to non-standard CSS selectors.

Other issues

[x] Major - <th> tags should have "id" or "scope" attributes
[x] Major - 'this.dataClassRecords' is assigned to itself.
[x] Minor - <strong> and <em> tags should be used (relates to fontawesome icons)
[ ] Minor - <table> tags should have a description

After these, review any remaining issues left.

Remaining issues are "Add a description to this table", which are classified by Sonarqube as minor bugs. Agreed that these can be sorted out another time.


Windows Container Support

 Freshdesk Ticket ID: 232
 Freshdesk Ticket Agent: Dale Georg
 Freshdesk Ticket Agent Email<EMAIL_ADDRESS> Ticket Priority: Low
 Freshdesk Ticket Description:

John Karabaic(assign)

May 20, 2020 17:10

To: Steve Jacobs Show all

Battelle is using some data acquisition software that runs only on Windows.May need first-class support for windows to extract data from it, if there aren't any api's available.Due date represents the checkin time for this requirement, when we'll write the feature request.Please add any technical details about the software (vendors, API's, etc) that'll help.

Agent Dale Georg linked Freshdesk ticket 232 for this issue.


Weird code block rendering

Version<EMAIL_ADDRESS><EMAIL_ADDRESS>Tested browsers:

Firefox 61.0.2 (64-bit)
Chromium 68.0.3440.106 (64-bit)

Markdown source code
This markdown is extracted from info.description

``` text
  Link: </protocoles/18/sections/34> rel=backlink; title*=UTF-8'fr'Accident%20d%27exposition%20au%20sang,
        </techniques/65/sections/45> rel=backlink; title*=UTF-8'fr'Valve%20ASHERMAN
```

Expected visual output

This visual output disappears after few milliseconds.
Observed visual output

Observed DOM
<code class=" language-json">Link: </protocoles/18/sections/34> rel=backlink; title*=UTF-8'fr'Accident%20d%27exposition%20au%20sang,
      </techniques/65/sections/45> rel=backlink; title*=UTF-8'fr'Valve%20ASHERMAN
</code>

My guess is that a class is removed with some sort of sanitizing process, resulting in an ugly code block.

Should be fixed with<EMAIL_ADDRESS>Could you verify please?

@RomanGotsiy Yes I can confirm, the issue has vanished! Thanks a lot.


[Feature Request] Add C# Code Generation

🚀 Feature Request
Add code generation support for C#.
Motivation
C# is a very popular language, which would be nice to support. I need support for it for my own project.
Pitch
C# is rather similar to Java, so implementing it is a fairly straightforward port of the existing Java code generator.
I'm currently working on implementing this here: https://github.com/mattico/serde-reflection/tree/csharp
I would like to upstream this support if there is a desire for it.

Thanks for the feature request. C# has been on our radar for some time. I will be happy to review a PR based on the Java codegen.


SWARM-1906: JAX-RS Fraction does not introspect jar files inside war files when searching for the presence of ApplicationPath

Motivation
I noticed that there were multiple JAX-RS applications deployed for my WAR file. As I debugged further, I realized that the issue was because the JAR file (inside the WAR file) was being viewed as a ByteArrayAsset and was being skipped over. With the recent fixes to IndexView representation, the fix was actually quite simple.
Modifications

During the deployment process, query the IndexView to see if there are any classes which contain the @ApplicationPath annotation.
If present, do not generate the default JAX-RS Application class.

Result

Verified a locally patched build and ensured that things are working as desired.
Ensured that the build has no errors.

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you created a JIRA and used it in the commit message?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

add to whitelist

As mentioned on the issue itself, this isn't the fix we want to implement.
What we want to do is defined in SWARM-1471, which is to make the default Application generation optional. With that change this one is not necessary

@kenfinnigan I see that you are recommending the reorganization of the logic for creating the default JAX-RS application class. I can definitely close this PR in favor of the other solution. The concern that I see with SWARM-1471 is that we may end up breaking away from the spec.
If there are no definitions, then the framework should automatically create a default Application instance.
So I am not sure if that would be the right way to achieve it. What do you think?

What we did with WF Swarm by generating that class actually goes against the Java EE spec and caused much confusion.
That's why we need to simplify and only generate the Application class when the developer sets a property to request it

@kenfinnigan Ah! Understood! So basically, WF Swarm provides the ability to declaratively define the JAX-RS application root. That is neat!
I shall close this PR and raise a separate one. This behavior has confused me as well 😄.

@kenfinnigan Quick question ... The fix for 1471 is a breaking change. Do you still want to proceed with it? Or is it used rarely that the impact would be rather low?

To fix it we're going to have to break the behavior at some point

Closing this PR. Will raise one that fixes the issue identified in SWARM-1471.


docs(security) ingress auth using zone token

should we use a common term for zone ingress / zone egress?
zone proxy maybe?

@jakubdyszkiewicz what's next for this PR?

@jakubdyszkiewicz seems like you need to update this PR

refreshed for the jekyll rework


adds --url args to every wp search-replace to make it work with multisites

Solves #20

Hi there!
Since it seems that @lavmeiker is not active anymore (at least for now) we just mirrored this repo and decided to continue maintaining it.
If you don't mind, just set up your pull request with our repo and we will merge it most certainly. =)
Thanks!
https://github.com/jshmrtn/capistrano-wpcli


fix Windows SDK id in installation instructions

@compnerd Saleem FYI @tristanlabelle also created a pull request for the same issue.

Thanks @ktraunmueller - I saw that later. I think that we should be able to resolve the conflicts (if any) and get both merged.


Approach to implementing user roles

We need to agree our approach to implementing User Roles in the digital platform.
[This ticket will be updated with the approach we will take.]

@mbrookeswebdev the following links are worth a read with regards how we could implement user roles in the Admin UI.
Global helpers/plugins in Vue:
https://medium.com/js-dojo/registering-globally-your-helpers-in-a-vue-app-b7ba3b9596b3
https://vuejs.org/v2/guide/plugins.html
Examples of what others do:
https://vuejsdevelopers.com/2018/01/08/vue-js-roles-permissions-casl/
https://medium.com/@aimme/very-simple-way-to-manage-permissions-on-vue-js-6ecbf930c0aa
https://casl.js.org/v5/en/package/casl-vue

@mbrookeswebdev the following links are worth a read with regards how we could implement user roles in the Admin UI.
Global helpers/plugins in Vue:
https://medium.com/js-dojo/registering-globally-your-helpers-in-a-vue-app-b7ba3b9596b3
https://vuejs.org/v2/guide/plugins.html
Examples of what others do:
https://vuejsdevelopers.com/2018/01/08/vue-js-roles-permissions-casl/
https://medium.com/@aimme/very-simple-way-to-manage-permissions-on-vue-js-6ecbf930c0aa
https://casl.js.org/v5/en/package/casl-vue

I've created a plugin that checks if a user has a permission, here is the link to the branch.

I've created a plugin that checks if a user has a permission, here is the link to the branch.

I'd say this task is done. The above approach / outline can be included in the Epic ticket, say, and used for implementation tasks.
Have moved back to 21JANA milestone as that's where the majority of the work happened.

There is a branch in Admin repo with some prototype code written for ticket 1147.


Guide user through publishing process

The sidekick allows a author to click on the "Live" or "Production" button before the page has been published. This leads to a cached 404 in the author's browser cache which is likely stuck there for 2h and creates a lot of confusion.
We should explore:

disabling (or hiding) the "Live" and "Production" buttons as long as a page isn't published
force-updating the browser cache on "Publish" before redirecting to the production URL (like we do on "Preview" from the editor or "Reload" from the preview)

cc @davidnuescheler @tripodsan


erigon: 2.37.0 -> 2.38.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for erigon is: Ethereum node implementation focused on scalability and modularity
meta.homepage for erigon is: https://github.com/ledgerwatch/erigon/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.38.1 with grep in /nix/store/g5bljg9a10qqqx6y4dy9al6hw0vbapj8-erigon-2.38.1
found 2.38.1 in filename of file in /nix/store/g5bljg9a10qqqx6y4dy9al6hw0vbapj8-erigon-2.38.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
erigon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g5bljg9a10qqqx6y4dy9al6hw0vbapj8-erigon-2.38.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A erigon https://github.com/r-ryantm/nixpkgs/archive/759bf66c38900482d043a91b6ff6541a2e7bfa4a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g5bljg9a10qqqx6y4dy9al6hw0vbapj8-erigon-2.38.1
ls -la /nix/store/g5bljg9a10qqqx6y4dy9al6hw0vbapj8-erigon-2.38.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

erigon

Maintainer pings
cc @d-xo @happysalada for testing.

@ofborg build erigon

Result of nixpkgs-review pr 215209 run on aarch64-darwin 1

1 package built:

erigon


Revert "[Backport 2.2] Add new settings for SAML and OIDC that allow for cookie splitting"

Reverts opensearch-project/documentation-website#3889. This is only released in 2..7.

This is only released in 2.7.


Looking for an specific expression

First, congrats @mvfsillva! Awesome and fun project 👏👏👏
I was wondering if we can evolve the API in a way we can search for an specific expression.
I mean, there's some expressions which are used in different regions and sometimes it have the same meaning and sometimes don't.
It'll be nice if the API provides the ability of discovering.
Examples:
Request
curl https://dialetus-service.now.sh/search/bah
Response
]
  {
    "region": "gauches",
    "dialect": "Bah",
    "meanings": ["Tapada", "Lento", "Disperso"],
    "examples": ["Bah, sério?!"]
  }
]

Request
curl https://dialetus-service.now.sh/search/baita
Response
]
  {
    "region": "gauches",
        "dialect": "Baita",
        "meanings": ["Grande", "Imenso"],
        "examples": ["Bah tchê, que baita de um problema tu arranjaste?!"]
  },
  {
    "region": "paulistes",
        "dialect": "Baita",
        "meanings": ["Grande", "Imenso"],
        "examples": ["Meu, que baita de um problema!"]
  }
]

With this approach we can even provide a full search functionality:
Request
curl https://dialetus-service.now.sh/search/ba
Response
]
 {
    "region": "gauches",
        "dialect": "Bah",
        "meanings": ["Tapada", "Lento", "Disperso"],
        "examples": ["Bah, sério?!"]
  },
  {
    "region": "gauches",
        "dialect": "Baita",
        "meanings": ["Grande", "Imenso"],
        "examples": ["Bah tchê, que baita de um problema tu arranjaste?!"]
  },
  {
    "region": "paulistes",
        "dialect": "Baita",
        "meanings": ["Grande", "Imenso"],
        "examples": ["Meu, que baita de um problema!"]
  },
  {
    "region": "paulistes",
         "dialect": "Bagulho",
         "meanings": ["Qualquer tipo de objeto (lapis, papel, celular)"],
         "examples": ["Empresta esse bagulho ai pra mim?", "Passa essa bagulho ai mano!"]
  }
]

Or even better, we could group by regions in the response:
]
 {
    "region": "gauches"
        "dialects": [
         {
           "dialect": "Bah",
           "meanings": ["Tapada", "Lento", "Disperso"],
           "examples": ["Bah, sério?!"]
          },
         {
          "dialect": "Baita",
          "meanings": ["Grande", "Imenso"],
          "examples": ["Bah tchê, que baita de um problema tu arranjaste?!"]
         }
    ]
  }
]

This project and idea is so rich! That's a lot of applications and ways to scale.
E.g.: Integrate with dictionaries, etymology, audio expressions, and so on.
Cool!
Have you considered GraphQL?
Perhaps it could be a good alternative to amplify the service even more 🙂

First, thank you @jaydson, I'm a fan of the Brazilian language culture and I really want to evolve this project!
About your proposals:
1 - I think it's completely valid, let's implement it?
2 - Yes, it will be nice to change for GraphQL.
What do you think?

I link this issue, I think this is very interesting. Can I implement it?
But I have one point of disagreeing with the proposal, it's about the global search
curl https://dialetus-service.now.sh/search/bah

I think it's better to follow up on the model of Google and Twitter APIs, like this:
curl https://dialetus-service.now.sh/search?q=bah

When q is the query parameter, so we can do things like this:
curl https://dialetus-service.now.sh/search?q=bah+bagulho

So in the future, this project can have other things like:
locale filter to help with to set the state.
** count**, limit and ** offset** to improve the response as a pagination list.
Twitter Search API
Google Search API

Hi, @malaquiasdev I liked this!
Would you like to implement?

Yep! assign to me please S2.


Fix #2213 - Workaround for TFS dependency resolution

I tested this solution for TFS, VSTS and NuGet.org - all work fine now

cool!


Add brew metadata as meta tags into head on share render

Feature for v3: Some the metadata fields: title, description, and thumbnail should be placed within meta tags on the render of the share page.

https://github.com/naturalcrit/homebrewery/blob/master/client/template.js has been updated to include the brew title into the <title> element.
The brew is passed in via props, so nothing elsewhere needs updating.
There are some other html meta tweaks that need doing here too for proper validation.

The top of the template could look more like this:
<!DOCTYPE html>
<html lang="en">
<head>
		<meta charset="utf-8">
		<title>${title.length ? `${title} - The Homebrewery`: 'The Homebrewery - NaturalCrit'}</title>
		<link href="//use.fontawesome.com/releases/v5.15.1/css/all.css" rel="stylesheet" />
		<link href="//fonts.googleapis.com/css?family=Open+Sans:400,300,600,700" rel="stylesheet" type="text/css" />
		<link href=${`/${name}/bundle.css`} rel='stylesheet' />
		<meta name="description" content="${props.brew.description}">
		<meta name="author" content="${props.brew.authors.join()}">
		<link rel="icon" href="/assets/homebrew/favicon.ico" type="image/x-icon" />

So .. add a lang="en", add a charset meta, add description meta, add author meta.
Eventually also props.brew.tags in some form just for completeness. (I'm unsure what html element to use there.)

The lang="en" for the share page should remain as en because the share page has english language UI elements.
Later, once the brew meta data includes an author specified lang option that would be inserted into the <div class="pages> element (i.e. not the outermost <html> element.)

With #1777 and #2080 merged now, I think this issue just needs the user selectable language attribute set - I imagine we can do this in a very similar way to the metadata template changes.

Can this issue be closed?  The only remaining thing left here is the language tag, which is better covered by the issue (and attached PR) #1343 .

The lang attribute in the <html> element is for the document as a whole, which includes any UI elements. Generally, it should get set to en (because that is what we're using everywhere) .. but later when a user can change a user setting for the UI lang then it would go there.
However, we also now have brew.lang, which will be applied to the div that wraps the rendered brew.
This brew.lang (#1343) should probably also be applied to these elements in the head

<meta property="og:title" content="¡Hola! La leyenda de la siesta." lang="es">
<meta property="og:description" content="Aquí está mi hermosa cerveza." lang="es">
<title lang="es">¡Hola! La leyenda de la siesta.</title>

But not this:

<meta property="og:site_name" content="The Homebrewery - Make your Homebrew content look legit!">
(not until we change the html document lang via user preferences)

With #1343 closed as completed, this issue is completed as well.


Psc-filing-api tech debt

Replace Companies House Parent version to 2.1.6
Upgrade Spring Boot Version to at least 3.3.2
Make sure the Jib plugin docker images are correct
Implement the new vulnerability suppression method

Jira:  JU-532

CI: No security warnings found

CI: Security warnings found!

CI: No security warnings found

CI: Security warnings found!


test

What would you like to share?
test
Additional information
No response

hey @eddiejaoude can u be specific about what i should check and can u assign it to me?

@VarshaUN I am testing the project which reads the GitHub API for issues with the label "good first issue" so this is only a test issue - it is not to be assigned to anyone

I would like to contribute to your project maybe when issues are available @eddiejaoude .

Sure 👍
Sorry for the confusion on this issue


release overwrites index.html

I tend to shy away from modifying files that are under source control during a release. Maybe instead of modifying index.html inline, we copy it into dist and make everything relative to that?

Ya, I was on the fence about that too.  For me, the only thing that does this is the CI and when I'm testing but its not optimal.  Ya, I was thinking it might be best to do a index.dist.html or something or maybe put it in the dist?

Yea, a index.tpl.html is a good enough solution for now and then add a .gitignore for index.html. Putting it in dist would be difficult because it would require changing paths to things in a lot of places.
When I do build systems, I generally try to just make a folder like dist or target or build and then build everything in there and make everything relative to that folder. It makes doing the clean and packaging tasks a lot simpler because everything I need is in one place.

closing since #20 has a better fix idea


Hidden text in the email box of the subscribe button .

When you click to enter your email in the box before the subscribe button. The text is hidden and the characters are not shown.

Hi @Rihem-taboubi I tried to resolve the issue can you pls assign it to me and check the PR
https://github.com/akshitagupta15june/PetMe/pull/350

Hello, @Akshaykumar4524 I couldn't assign you.I am not the owner of this repository.

@akshitagupta15june can you please have look into the PR


Remote I/O error

Hey guys,
I've been trying to get this sensor working on a pi but for some reason I keep getting this remote I/O error.
Hardware:

RPI 3B V1.2
STS35

Wiring:

STS35 VCC - RPI 5V pinout (tested the voltage - 4.01V)
STS35 GND - RPI GND pinout
STS SDA - RPI SDA pinout
STS SCL - RPI SCL pinout

Log printout:
Traceback (most recent call last):   File "get_temperature_period.py", line 45, in <module>     setup()   File "get_temperature_period.py", line 26, in setup     sensor.begin()   File "/home/pi/DFRobot_STS3X/python/raspberrypi/DFRobot_STS3X.py", line 157, in begin     self.sensor_reset()   File "/home/pi/DFRobot_STS3X/python/raspberrypi/DFRobot_STS3X.py", line 264, in sensor_reset     self.sensor_break()   File "/home/pi/DFRobot_STS3X/python/raspberrypi/DFRobot_STS3X.py", line 258, in sensor_break     self._write_command(STS3X_CMD_MEA_PER_BREAK_MSB, [STS3X_CMD_MEA_PER_BREAK_LSB])   File "/home/pi/DFRobot_STS3X/python/raspberrypi/DFRobot_STS3X.py", line 410, in _write_command     self._i2c.write_i2c_block_data(self._i2c_addr, cmd_msb, cmd_lsb) IOError: [Errno 121] Remote I/O error
i2cdetect -y 1 printout:

I know for a fact that the sensor is working since I tested it on a Arduino Leonardo.
Can someone please point me in a direction?

Hey guys,
From the picture, the address you scanned is 0x4b, which is different from the default 0x4a in Demo. Have you tried to change the address of module i2c in Demo:

# Determine I2C address according to the ADDR pin pull-up or pull-down
# ADDR pin pull-down: STS3X_I2C_ADDRESS_A   0x4A
# ADDR pin pull-up: STS3X_I2C_ADDRESS_B   0x4B
sensor = DFRobot_STS3X(i2c_addr = STS3X_I2C_ADDRESS_A,bus = 1)

I feel like such a idiot... you are correct.
From reading the specs on your site I took that 0x4B would be used by default

but I should've just used my eyes xD
Thanks @qsjhyy

It was really an oversight on my part. I think I subconsciously took the letter A as the default. I've changed the default address used for the Python examples for now, because it's really misleading.
Thanks @ParcoMolo


Account Compression TS SDK: add MerkleTree static functions

Additions:

Easier to use constructor method sparseMerkleTreeFromLeaves which by default creates sparse MerkleTree leaves for you
Easier to use verify & hashProof methods (now static)

Fix:

Move js-sha3 to dependencies, instead of devDependencies

@jnwng MerkleTree.sparseMerkleTreeFromLeaves should be used to create trees instead of the normal constructor. The normal constructor is exposed for testing purposes, and for those that are familiar with the MerkleTree data structure.


[PL-40533] Custom Certificate Management

PL-40533
EPIC: https://harness.atlassian.net/browse/PL-40208
Enhancement: https://harness.atlassian.net/browse/PL-29964
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screenshot.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: saifeemustafaqharness:x: pratmitYou have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65df12e5414eff07d21fd89e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65e0584ed76367f2ee6eac0d--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65e059e8c9dd82f4599a46a0--harness-developer.netlify.app

@pratmit @brian-f-harness Ready for review

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65fbdf51b19da7847678bce1--harness-developer.netlify.app